-
Notifications
You must be signed in to change notification settings - Fork 27
Conversation
Can one of the admins verify this patch? |
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.
Just one question, otherwise LGTM, very nice work, glad you were able to track this down.
@@ -429,6 +429,7 @@ def make_release_pr(self, new_pr): | |||
name, email = self.get_user_contact() | |||
repo.set_credentials(name, email) | |||
repo.set_credential_store() | |||
repo.checkout('master') |
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.
Could you please elaborate why is this needed? Two lines below there is another checkout. Ideally please include a comment in the code.
[test] |
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.
lgtm
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.
LGTM, thank you!
This solves the bug #173
make_new_pypi_release() will now check back to master hence will have no side effect to other functions.
make_release_pr() will additionally checkout for master again because its important that the current branch is master before release pr branching
make_release_pr() will also check back to master after doing the release_pr