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: Issues should be added to Examine for better performance #357

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

Conversation

abjerner
Copy link
Contributor

Still very much work in progress 😄

I only had a brief moment to work on this, so the first step is changing a bit in the way issues are loaded from disk. This is partially to utilize my GitHub package a bit more, as we then have to implement less logic for Our.

- The RepositoryManagementService and Issue now relies on manual parsing the JSON, as this gives us some more control
- As a result of above, Issue is now mostly just a wrapper for GitHubIssue from my GitHub package (means less logic for us to handle in this repo)
- Comments and Events are now lazy loaded as we may not always need them
- Added new PullRequest class, which is currently just an empty class inheriting from the Issue class
@nul800sebastiaan nul800sebastiaan changed the base branch from master to main July 15, 2020 13:25
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