Repository navigation
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8a93f06c-3318-405d-ad86-fcf729e58207
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
The deployment workflow directly interpolates artifact_version into shell source, enabling command execution with repository and Pages write permissions before validation.
3 open issues
What changed in this PR
Separates Java dependency coordinates from documentation versions so preview sites can reference rolling Maven snapshots while preserving release-specific site identity.
Changes:
- Adds an optional
artifact_versiondeployment input and Maven property. - Prepares dependency examples and conditionally adds snapshot repository instructions.
- Adds rendering tests and a guard intended to require artifact versions for preview deployments.
| File | Description |
|---|---|
src/site/markdown/index.md |
Uses artifact coordinates and repository placeholders in installation examples. |
src/site/markdown/getting-started.md |
Updates setup examples for rolling snapshot artifacts. |
scripts/prepare-site-versions.test.mjs |
Tests preview and stable preparation plus base-version validation. |
scripts/prepare-site-versions.mjs |
Validates versions, rewrites coordinates, and renders repository instructions. |
pom.xml |
Defines an independently overridable artifact version. |
.github/workflows/deploy-site.yml |
Accepts the artifact version and passes it through site preparation and rendering. |
Review effort: Balanced (auto)
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8a93f06c-3318-405d-ad86-fcf729e58207
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8a93f06c-3318-405d-ad86-fcf729e58207
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8a93f06c-3318-405d-ad86-fcf729e58207
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Superseded by #249 .
Problem
Versioned preview documentation currently uses the shared SDK prerelease as the Maven, Gradle, and JBang coordinate. Java previews now use one rolling Maven snapshot per base version, so those examples would point to an unpublished coordinate.
Changes
artifact_versionworkflow input while preservingversionfor site directories and release/source identity.Validation
node --test scripts/prepare-site-versions.test.mjs— passed (3 tests).mvn site -Dsite.version=1.0.15-preview.2 -Dartifact.version=1.0.15-SNAPSHOT ...— BUILD SUCCESS; rendered site kept1.0.15-preview.2metadata and used1.0.15-SNAPSHOTplus snapshot repositories.mvn site -Dsite.version=1.0.15 -Dartifact.version=1.0.15 ...— BUILD SUCCESS; rendered stable coordinates remained1.0.15with no snapshot repository.Rollout dependency
Merge this pull request before github/copilot-agent-runtime#33343 begins sending
artifact_version. The runtime pull request remains draft until this receiving workflow is available.