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

Refactor GitHub.Event logic #1095

Merged
merged 1 commit into from
Oct 19, 2017
Merged

Refactor GitHub.Event logic #1095

merged 1 commit into from
Oct 19, 2017

Conversation

joshsmith
Copy link
Contributor

@joshsmith joshsmith commented Oct 19, 2017

What's in this PR?

  • Split Event.IssueComment out into Sync.Comment

@joshsmith joshsmith force-pushed the refactor-events branch 2 times, most recently from 4df0687 to 064e6cc Compare October 19, 2017 22:46
Split Event.IssueComment out into Sync.Comment

Split Event.Issues out into Sync.Issue

Move RepoFinder into Sync.Utils

Split Event.PullRequest out into Sync.PullRequest

Improve docs

Refactor tests for issues event and issue sync

Refactor tests for issues comment event and comment sync

Refactor tests for pull request event and pull request sync
@joshsmith joshsmith merged commit ef62b5d into develop Oct 19, 2017
@joshsmith joshsmith deleted the refactor-events branch October 19, 2017 23:15
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.

None yet

1 participant