-
Notifications
You must be signed in to change notification settings - Fork 995
Open
Description
The facade contributor resolution process (augur/tasks/github/facade_github/tasks.py process_commit_metadata) is essentially written solely for github, making it nearly impossible to implement for gitlab, etc without introducing massive code duplication.
heres my migration/rearranging plan for this component:
- migrate all the github url assembly and API calling/processing to the existing GitHubDataAccess class.
- Create a new data class
AugurForgeUserto act as an internal model/representation of a generic forge-independent user that can be adapted for writing to our db - make sure GitHubDataAccess can return in this format - define an interface for the API methods that are needed for contributor resolution, allowing us to use OOP principles to swap out GitHubDataAccess with other classes when we go to add more forges, keeping the remaining resolution logic
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels