-
Notifications
You must be signed in to change notification settings - Fork 485
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
update Making Releases in dev guide (hotfixes too) #8795 #8877
Conversation
|
||
Because of the hotfix version, any SQL scripts in "develop" should be renamed (from "5.11" to "5.11.1" for example). To read more about our naming conventions for SQL scripts, see :doc:`sql-upgrade-scripts`. | ||
|
||
Please note that version bumps and SQL script renaming both require all open pull requests to be updated with the latest from the "develop" branch so you might want to add any SQL script renaming to the hotfix branch before you put it through QA to be merged with develop. This way, open pull requests only need to be updated once. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[possibly a dumb] Question - is this actually necessary, to go to the trouble of renaming flyway scripts on account of hotfix/emergency releases? In our current situation, we could have kept the existing V15.11.0.* scripts in the develop branch, and it wouldn't have broken anything, I don't think. Are there any situations where it would in fact be absolutely necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Renaming the SQL scripts is not strictly necessary but it keeps things tidy and consistent, which I think is worth it.
(That's why I wrote it this way and did the renaming.)
I say we try it. Try to be tidy and consistent. If it's really a pain point we can drop it some day.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't imagine this ever becoming a real pain point. Hotfix releases are rare by nature.
And sure, "tidy and consistent" should be a good enough reason.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good stuff in this pr!
tiny cosmetic change to the version number in the example
At standup this morning we agree that it's fine to go ahead and merge this. I'm happy with it and @landreev took a look. We can always make another PR with further revisions later. |
What this PR does / why we need it:
We've never made a hotfix before. Let's document the plan and revise as needed. I also addressed #8795 and did some general cleanup including moving the part about release notes to the top since we always write release notes first.
Which issue(s) this PR closes:
Special notes for your reviewer:
You can preview the HTML at https://dataverse-guide--8877.org.readthedocs.build/en/8877/developers/making-releases.html
Suggestions on how to test this:
Just docs.
Does this PR introduce a user interface change? If mockups are available, please link/include them here:
No.
Is there a release notes update needed for this change?:
No.
Additional documentation:
None.