Add CWE support in multiple importers#1526
Conversation
Signed-off-by: ambuj <kulshreshthaak.12@gmail.com>
|
@ziadhany I don't know why some tests related to API are failing please check what should I do to resolve that. |
Signed-off-by: ambuj <kulshreshthaak.12@gmail.com>
ambuj-1211
force-pushed
the
add-cwe-support-in-multiple-importers
branch
from
229b70a to
131f37c
Compare
@ambuj-1211 I don’t see any errors in the test. Please let it fail in the CI so I can investigate the issue. |
Signed-off-by: ambuj <kulshreshthaak.12@gmail.com>
|
@ziadhany please have a look on this I have resolved the issues which you indicated earlier. |
ziadhany
left a comment
ziadhany
left a comment
There was a problem hiding this comment.
@ambuj-1211, you are doing a great job! Please make the required changes, and I hope we can merge this PR this week.
…nto add-cwe-support-in-multiple-importers
Signed-off-by: ambuj <kulshreshthaak.12@gmail.com>
…b.com/ambuj-1211/vulnerablecode into add-cwe-support-in-multiple-importers
|
@ziadhany please review the changes |
ziadhany
left a comment
ziadhany
left a comment
There was a problem hiding this comment.
@ambuj-1211, the code looks good. Just run the importers, print the logs, and then we can merge.
|
@ziadhany Here are the logs |
|
@ziadhany what's the blockers here ? |
|
@TG1999 Nothing, the pull request is ready to merge! |
4 participants
Fixes: #1093
This adds cwe data to following importers: