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

GitHub support #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

kenodegard
Copy link

Created an abstract Package class (ABCPackage) that contains most of the old Package functionality. The remaining old functionality of Package was placed in the subclass PyPIPackage.

This allowed for the creation of a GitHubPackage that mirrors PyPIPackage in functionality except instead of using PyPI as the source, GitHub is used.

From there both PyPIPackage and GitHubPackage are bundled together into the new Package class. This new Package class returns the applicable package class (PyPI vs GitHub) depending on the arguments passed (or by accessing the class methods).

Refactored logic such that Package becomes more dynamic in that it is now more like a framework for package handlers. Was able to move the majority of old code into an abstract class that is then inherited by PyPIPackage and GitHubPackage with the individual implementation specifics.
Moved a few items around and added unittesting.
Allow autoupgrade to be used without the github module being installed.
Included important information about PyGithub being required for Github features to work.
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.

1 participant