Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generate release notes from tag release to the built tag. #895

Merged
merged 5 commits into from
Feb 27, 2024

Conversation

nat-goodspeed
Copy link
Collaborator

@nat-goodspeed nat-goodspeed commented Feb 26, 2024

Explicitly specify both target_commitish and previous_tag inputs to the GitHub release-notes generator. Unless they're both explicitly passed, action-gh-release guesses wrong about the range of commits to consider for release notes generation.

This change uses the new action-gh-release feature previous_tag (secondlife-3p/action-gh-release#2). Once that PR is merged, we can revert to referencing secondlife-3p/action-gh-release@v1.

This behavior also depends on maintaining a new viewer tag release to track the most recent release, updating it every time we promote a new default download viewer.

Also add cross-links between the build page and the corresponding release page.

relnotes:
This text should be copied to the release page.

It should include everything after the relnotes: line.

Also try to cross-reference release page and build page.
Using github.ref as action-gh-release's target_commitish produces:

⚠️ GitHub release failed with status: 422
[{"resource":"Release","code":"invalid","field":"target_commitish"}]
This should (!) allow us to generate full release notes relative to the
previous viewer release, instead of letting action-gh-release guess
incorrectly.

Also try again to add to the release page a back-link to the specific build.
instead of on the current tag 7.1.2-release.
@nat-goodspeed
Copy link
Collaborator Author

Note that the build page has a link to the release page, and vice-versa. (Scroll down on the release page.)

This PR did not yet exist at the time of that tag build, which is why the relnotes: text wasn't found or copied.

@nat-goodspeed nat-goodspeed merged commit 77b51cf into DRTVWR-588-maint-W Feb 27, 2024
11 checks passed
@nat-goodspeed nat-goodspeed deleted the relnotes-w branch February 27, 2024 15:15
@github-actions github-actions bot locked and limited conversation to collaborators Feb 27, 2024
@nat-goodspeed
Copy link
Collaborator Author

I'll be interested to see the next tag build.

@marchcat
Copy link
Contributor

I'll likely build one tonight.

@marchcat marchcat added this to the [DRTVWR-588] Maintenance W milestone Feb 27, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants