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

Final-Release: use a more descriptive git tag #90

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

a-kenji
Copy link
Contributor

@a-kenji a-kenji commented Sep 14, 2024

Use a more descriptive git tag for the release.

The problem with a 24.05-style tag is, that they will land in peoples pinned sources - and those people will wonder why they never get any updates.

This changes hopes to make this less prevalent by adding a more descriptive name for the actual git tag: It tags the branch-off for the new channel release, but it is not the actual channel.

Use a more descriptive git tag for the release.

The problem with a `24.05`-style tag is, that they *will* land in peoples
pinned sources - and those people will wonder why they never get any
updates.

This changes hopes to make this less prevalent by adding a more descriptive
name for the actual git tag: It tags the branch-off for the new channel
release, but it is *not* the actual channel.
@wegank wegank linked an issue Sep 29, 2024 that may be closed by this pull request
@Mic92
Copy link
Member

Mic92 commented Oct 30, 2024

cc @RossComputerGuy

@@ -141,7 +141,7 @@ export NEWVER=23.05
1. Find the commit id and tag the release **on the release branch**:

```bash
git tag --annotate --message="Release $NEWVER" $NEWVER <COMMIT_ID>
git tag --annotate --message="Release $NEWVER" "branch-off-$NEWVER" <COMMIT_ID>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
git tag --annotate --message="Release $NEWVER" "branch-off-$NEWVER" <COMMIT_ID>
git tag --annotate --message="Release $NEWVER" "$NEWVER-branch-off" <COMMIT_ID>

I guess I'm preferring 24.11-branch-off to branch-off-24.11, but I'll let @RossComputerGuy decide.

Copy link
Member

Choose a reason for hiding this comment

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

I think branch-off-24.11 is better because we have release-24.11 so it would be consistent.

@RossComputerGuy RossComputerGuy merged commit 812bd5b into NixOS:master Oct 30, 2024
1 check passed
@wegank wegank mentioned this pull request Oct 30, 2024
@a-kenji a-kenji deleted the hide-foot-gun branch October 30, 2024 22:54
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.

Tag name is dangerous
4 participants