Skip to content

Release process#1292

Merged
softins merged 3 commits intojamulussoftware:masterfrom
softins:release-process
Mar 17, 2021
Merged

Release process#1292
softins merged 3 commits intojamulussoftware:masterfrom
softins:release-process

Conversation

@softins
Copy link
Copy Markdown
Member

@softins softins commented Mar 17, 2021

Document the process for doing a pre-release or release.

Copy link
Copy Markdown
Member

@hoffie hoffie left a comment

Choose a reason for hiding this comment

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

This is great, thanks for doing this! I have nothing to add :)

I guess this will also help shorten the release tracking checklist for the next iterations as we can just reference this document. :)


```
$ git tag r3_7_0
$ git push origin tag r3_7_0
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Have never seen or used this syntax, but seems correct based on git push --help.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

There may be other ways to achieve it via Github, but the above was what worked for me. It means the complete process can be achieved via the shell without visiting the website.

Copy link
Copy Markdown
Collaborator

@pljones pljones Mar 17, 2021

Choose a reason for hiding this comment

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

Also needs

git tag latest
git push tags --force

to move latest to the latest release. (origin is redundant, in the above, I think.)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ah yes, but only if it’s a formal release, not a pre-release, of course. I’ll update soon.

In line 93 above, I think origin is required, because <repository> is required to precede a <refspec>, which is what tag r3_7_0 is.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Reading the man page for git push, I think you meant git push --tags --force, but I think that could be slightly risky, and I would prefer the more specific git push --force origin tag latest

@pljones
Copy link
Copy Markdown
Collaborator

pljones commented Mar 17, 2021

https://github.com/softins/jamulus/blob/release-process/RELEASE-PROCESS.md
Yep, looks pretty good - great to have it written out.

@softins softins added this to the Release 3.7.0 milestone Mar 17, 2021
Copy link
Copy Markdown
Member

@ann0see ann0see left a comment

Choose a reason for hiding this comment

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

Yes. This looks ok.


## If this is a proper release, move the `latest` tag

This needs the `--force` option to overwrite the existing `latest` tag and move it to the current commit:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You could also delete and re-apply the latest tag.

@softins softins merged commit 862b320 into jamulussoftware:master Mar 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Triage

Development

Successfully merging this pull request may close these issues.

4 participants