-
Notifications
You must be signed in to change notification settings - Fork 53
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
DOC release refinements #116
Conversation
2. Bump the version defined in ``skops/__init__.py`` | ||
3. Git grep for any TODO's that need fixing before the release (e.g. | ||
1. Check and update the ``CHANGES.md`` | ||
2. For a major release, create a new branch with the name "0.version.X", e.g. |
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.
Technically, according to semantic versioning, this would be a "minor" release. I think many people use "major" the way you do here, I'm not sure if we want to strictly follow semver.
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.
Yeah we don't, the whole ecosystem doesn't so I don't think it's a big deal here.
5. Create a PR with all the changes and have it reviewed and merged | ||
6. Create a tag with the format "v0.version", e.g. "v0.2", and push it to the | ||
remote repository. Use this tag for releasing the package. If there is a | ||
minor release under the same branch, it would be "v0.2.1" for example. |
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.
Numbers below this need to be incremented (6->7 etc.). I think that replacing the numbers with #.
numbers them automatically.
Co-authored-by: Benjamin Bossan <BenjaminBossan@users.noreply.github.com>
Applied suggestions. |
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.
Left two minor nits. :')
2. For a major release, create a new branch with the name "0.version.X", e.g. | ||
"0.2.X". This branch will have all tags for all releases under 0.2. | ||
3. Bump the version defined in ``skops/__init__.py`` | ||
4. Git grep for any TODO's that need fixing before the release (e.g. |
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.
Can you explain this a little more? Maybe type an example command?
Please ignore if it doesn't make any sense but if I were a new maintainer I might've been confused.
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.
added this.
deprecations) | ||
4. Update the ``CHANGES.md`` | ||
5. Create a PR with all the changes and have it reviewed and merged |
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.
Maybe tell them to tag @ skops-dev/maintainers.
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.
we have this point already in this doc.
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.
@merveenoyan I should have addressed your comments here.
2. For a major release, create a new branch with the name "0.version.X", e.g. | ||
"0.2.X". This branch will have all tags for all releases under 0.2. | ||
3. Bump the version defined in ``skops/__init__.py`` | ||
4. Git grep for any TODO's that need fixing before the release (e.g. |
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.
added this.
deprecations) | ||
4. Update the ``CHANGES.md`` | ||
5. Create a PR with all the changes and have it reviewed and merged |
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.
we have this point already in this doc.
Some release process refinements.