Skip to content
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

moved 'upgrade' and 'upgrade_all' to commands/upgrade.py #263

Merged
merged 1 commit into from
Nov 4, 2019

Conversation

ptisserand
Copy link
Contributor

This pull request is the first one for issue #255

Only upgrade and upgrade_all commands have been moved in a separate file.

@ptisserand
Copy link
Contributor Author

Hi, any feedback about this pull request ?

@cs01
Copy link
Member

cs01 commented Nov 3, 2019

This looks good besides the merge conflict.

@cs01
Copy link
Member

cs01 commented Nov 3, 2019

A force push was recently done, so this PR will have to be rebased onto origin/master. Apologies for the inconvenience. See #270.

@ptisserand
Copy link
Contributor Author

ptisserand commented Nov 3, 2019

Hi, I have updated my branch but I don't get it why CI test is now failing.
I have tried master branch and failure is also present...

Seems to be related to #266

@cs01
Copy link
Member

cs01 commented Nov 3, 2019

That is a known issue being dealt with across the python ecosystem. See #266 and the linked issue at the bottom.

When #269 is merged the issue will be fixed. Your PR looks good though. I’ll merge it when the other fix lands. Thanks for your patience with all of this.

GH pypa#255: break commands.py into individual files.

commands: added common.py to remove circular import.

Fixed linter issues.
@ptisserand
Copy link
Contributor Author

I have pushed a rebased version.

Copy link
Member

@cs01 cs01 left a comment

Choose a reason for hiding this comment

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

Thanks, looks good

@cs01 cs01 merged commit da8dff8 into pypa:master Nov 4, 2019
@ptisserand ptisserand deleted the split-commands branch November 7, 2019 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants