Skip to content

Migrate github importer to importer improver model#642

Merged
TG1999 merged 1 commit into
aboutcode-org:mainfrom
TG1999:migration/github
Mar 25, 2022
Merged

Migrate github importer to importer improver model#642
TG1999 merged 1 commit into
aboutcode-org:mainfrom
TG1999:migration/github

Conversation

@TG1999

@TG1999 TG1999 commented Mar 9, 2022

Copy link
Copy Markdown
Contributor

Signed-off-by: Tushar Goel tushar.goel.dav@gmail.com

@TG1999
TG1999 force-pushed the migration/github branch from fb11c1a to 0f66ce3 Compare March 9, 2022 10:53

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

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

Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/importers/github.py
Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/importers/github.py Outdated
@TG1999
TG1999 force-pushed the migration/github branch from 5b6bb67 to 104db79 Compare March 16, 2022 08:47

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

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.

Comment thread vulnerabilities/helpers.py Outdated
Comment thread vulnerabilities/helpers.py Outdated
Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/package_managers_2.py Outdated
Comment thread vulnerabilities/package_managers_2.py Outdated
Comment thread vulnerabilities/package_managers_2.py Outdated
Comment thread vulnerabilities/package_managers_2.py
Comment thread vulnerabilities/package_managers_2.py Outdated
@TG1999 TG1999 changed the title migrate github importer to importer improver model Migrate github importer to importer improver model Mar 21, 2022
@TG1999
TG1999 force-pushed the migration/github branch from bcc79fc to d1bc81c Compare March 23, 2022 12:41
Comment thread vulnerabilities/importers/github.py

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

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

Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/importers/github.py Outdated
Comment thread vulnerabilities/importers/github.py
Comment thread vulnerabilities/importers/github.py Outdated
Migrate github importer to importer-improver model

Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
@TG1999
TG1999 force-pushed the migration/github branch from d1bc81c to b6aad46 Compare March 25, 2022 08:15
@TG1999
TG1999 merged commit 33e083c into aboutcode-org:main Mar 25, 2022
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.

2 participants