Skip to content
This repository was archived by the owner on Mar 11, 2025. It is now read-only.

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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant