-
Notifications
You must be signed in to change notification settings - Fork 262
DOC: update release docs to use try_branch.py
#386
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
Conversation
Also, remove some emphasis on outdated check scripts.
8b4c882
to
ed40f34
Compare
|
||
If you are already on a Windows machine, you could have done the manual | ||
command to build instead: ``python setup.py bdist_wininst``. | ||
|
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 feel like this makes sense to move to below the release commit, otherwise you end up with x.y.z.dev
binaries. I ended up doing this sequence:
- Update docs and
nibabel/info.py
on a PR branch - Merge PR in Github
- Pull
maint/2.0.x
, tag and push tag - Push to pypi
- Push to buildbots
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.
Ah yes, you're right. But I personally like to avoid pushing the tag for as long as possible because it can't be reversed after. So I suggest try_branch pushing to the buildbots after the release commit, but before pushing the release commit or the release tag.
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 checked the builds on the branch that had been merged in before tagging, but that's a good policy.
Add an introduction to the release procedure.
b066c9b
to
b09926c
Compare
These are working for nibabel, but not when we need a compiler.
OK to merge this one? |
LGTM |
DOC: update release docs to use `try_branch.py` Also, remove some emphasis on outdated check scripts.
Thanks - merging. |
DOC: update release docs to use `try_branch.py` Also, remove some emphasis on outdated check scripts.
Also, remove some emphasis on outdated check scripts.