Skip to content

Migrate istio importer #1059 - #1058

Merged
TG1999 merged 4 commits into
aboutcode-org:mainfrom
TG1999:migrate/istio
Jan 10, 2023
Merged

Migrate istio importer #1059#1058
TG1999 merged 4 commits into
aboutcode-org:mainfrom
TG1999:migrate/istio

Conversation

@TG1999

@TG1999 TG1999 commented Jan 2, 2023

Copy link
Copy Markdown
Contributor

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

@TG1999 TG1999 changed the title Migrate istio importer Migrate istio importer #1059 Jan 2, 2023

@TG1999 TG1999 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggestions:

Comment thread vulnerabilities/importers/istio.py Outdated
Comment thread vulnerabilities/importers/istio.py Outdated
Comment thread vulnerabilities/importers/istio.py Outdated
Comment thread vulnerabilities/importers/istio.py Outdated
Comment thread vulnerabilities/importers/istio.py Outdated
Comment thread vulnerabilities/tests/test_data/istio/istio-expected.json Outdated
Comment thread vulnerabilities/tests/test_data/istio/istio-expected.json Outdated

@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.

LGTM with a few namespace adjustments

Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
@TG1999
TG1999 force-pushed the migrate/istio branch 3 times, most recently from c3aead0 to 14e0142 Compare January 10, 2023 12:13
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
@TG1999
TG1999 merged commit 2903edc into aboutcode-org:main Jan 10, 2023
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