Skip to content

ci: skip visualization build in downstream check - #2798

Merged
rsynek merged 1 commit into
mainfrom
ci/downstream-skip-visualization
Nov 2, 2026
Merged

rsynek merged 1 commit into
mainfrom
ci/downstream-skip-visualization

Conversation

@rsynek

@rsynek rsynek commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Downstream repositories received visualizations that require additional setup or skipping that part of the build. For the purpose of this CI, I am for the latter.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

Inaccessible downstream repositories prevent confirming that their Maven configuration consumes skip.visualization to disable all visualization setup while keeping backend verification enabled.

No open issues 🎉

What changed in this PR

This PR updates downstream CI to skip visualization builds, which require setup outside the scope of this check.

Changes:

  • Adds -Dskip.visualization to the downstream Maven build command, leaving the other options unchanged.
File Description
.github/​workflows/​downstream_build.yml Skips visualization builds during downstream verification.

Review effort: Balanced

@rsynek
rsynek merged commit 7a7eddf into main Nov 2, 2026
41 of 43 checks passed
@rsynek
rsynek deleted the ci/downstream-skip-visualization branch November 2, 2026 15:29

This branch was successfully deployed

1 active and 1 inactive deployments
documentation (preview) — b7dbb150 Deployed Nov 2, 2026 by rsynek via Build Documentation #841
internal — b7dbb150 Deployed Nov 2, 2026 by rsynek via approval_required #841
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants