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

WIP: Implement 'Links' functionality #337

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MartinFalatic
Copy link
Contributor

This allows PyUp to retrieve and display links for a given package using
the new package_metadata PyUp API.

This provides greater parity with the Github Enterprise PyUp.

@MartinFalatic MartinFalatic force-pushed the dev-mff branch 2 times, most recently from 52106c2 to 6850742 Compare March 18, 2019 20:28
This allows PyUp to retrieve and display links for a given package using
the new `package_metadata` PyUp API.
This provides greater parity with the Github Enterprise PyUp.

Also added a dry run option to make live testing easier to accomplish
@rafaelpivato
Copy link
Contributor

Interesting work you made here. I would urge you to bring the dry-run as a separate pull-request to close #372 maybe. Even though that is not required, it would speed up discussing and merging separate subjects.

Regarding the "links" idea, not sure I follow completely, but it sounds like a nice addition to the PR information. Is this still a work in progress? We made some changes on master, so you would need to rebase this as well.

Looking forward your feedback! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants