-
Notifications
You must be signed in to change notification settings - Fork 198
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
Migrate github importer to importer improver model #642
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I have a few nits for your consideration
5b6bb67
to
104db79
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here are some nits for your review... Please fix the ones you think are essential and create issues for the ones that can wait and merge at you convenience.
bcc79fc
to
d1bc81c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here are a few nits for your consisderation
Migrate github importer to importer-improver model Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
d1bc81c
to
b6aad46
Compare
Signed-off-by: Tushar Goel tushar.goel.dav@gmail.com