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

Add tag for Lemmy 0.19.2 #215

Closed
linux-cultist opened this issue Jan 11, 2024 · 9 comments · Fixed by #213
Closed

Add tag for Lemmy 0.19.2 #215

linux-cultist opened this issue Jan 11, 2024 · 9 comments · Fixed by #213

Comments

@linux-cultist
Copy link

linux-cultist commented Jan 11, 2024

Deploying the main version gets us 0.19.2 of Lemmy but just needs a tag added here in the repo as well.

@linux-cultist linux-cultist changed the title Upgrade to Lemmy 0.19.2 Add tag for Lemmy 0.9.2 Jan 13, 2024
@ticoombs ticoombs changed the title Add tag for Lemmy 0.9.2 Add tag for Lemmy 0.19.2 Jan 21, 2024
@Nothing4You
Copy link
Contributor

0.19.3 at this point

@Nothing4You
Copy link
Contributor

the tags are there (now, not sure about before), but they're not promoted to github releases

@codyro
Copy link
Collaborator

codyro commented Jan 22, 2024

I believe he's referring to the 1.3.1 tag. It got tagged before the VERSION file got bumped to the latest release, so it's behind.

@ticoombs can you retroactively include/update that in the tagged release? What is the best way to handle this? IIRC, it got bumped a day or two after 1.3.1. In that case would we do 1.3.2 anyway?

@dessalines
Copy link
Member

Side point, but that VERSION file gets bumped when we do a release: https://github.com/LemmyNet/lemmy/blob/main/scripts/release.sh#L24

Is that the right way to handle it, or should I be doing it differently since we do tagged releases here?

@linux-cultist
Copy link
Author

I think every time you update the VERSION, it should be followed by a new tag also. So 1.3.1 in this repo would be VERSION 0.19.1 of Lemmy, 1.3.2 would be VERSION 0.19.2 and so on.

Then it's possible to easily go back to previous version (unless there has been database upgrades between versions, which should be highlighted in the release notes).

@ticoombs
Copy link
Collaborator

ticoombs commented Jan 26, 2024

Is that the right way to handle it, or should I be doing it differently since we do tagged releases here?

Updating the VERSION file is fine, but creating the TAG is what causes the problem when people run git checkout $(git describe --tags). As this only gets the latest TAG.

The reasoning behind the 1.x vs 0.x tagging is because we (lemmy-ansible) also have to include upgrade notes and we didn't want to re-push versions, and the lemmy versions don't make sense when a system also contains ~4 other applications.

If you could not automatically create the tag @dessalines for lemmy-ansible that would solve the confusion.

<3


@ticoombs can you retroactively include/update that in the tagged release? What is the best way to handle this? IIRC, it got bumped a day or two after 1.3.1. In that case would we do 1.3.2 anyway?

@codyro You don't retroactively update the tags/versions, you push a new one. You are correct if the VERSION gets updated we need to also push a new tag/release.
The process is to:

  • create PR & get it merged with the new release notes / upgrade notes
  • git checkout origin main && git tag $version && git push origin $version
  • go into github releases, select the new $version, select the old $version, click generate release notes
  • edit release notes, if there is something extra. Select Publish release

@dessalines
Copy link
Member

I really appreciate yalls help in handling these releases.

@ticoombs
Copy link
Collaborator

@dessalines Is it possible to not tag the new releases for lemmy-ansible? That will avoid future issues & clashes with the tags/releases.

Updating main would be perfect & if I am reading the release.sh script correctly, skipping line 29 & 30 should be all the changes that is needed. That way lemmy-ansible still gets updated inline with the new Lemmy versions, while also allowing us to schedule our own releases/tags.

Thanks.

@dessalines
Copy link
Member

Sure, I'll remove that part from our lemmy deploy script now.

dessalines added a commit to LemmyNet/lemmy that referenced this issue Jan 30, 2024
Nutomic pushed a commit to LemmyNet/lemmy that referenced this issue Jan 31, 2024
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 a pull request may close this issue.

5 participants