-
Notifications
You must be signed in to change notification settings - Fork 201
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
Add license_url for GitHub Importer #656
Comments
hi , i want to work on this issue. could you please tell how to proceed? |
@iks1 |
Hi @ziadhany I have gone through the documentation but it is not clear to me that where the 'license_url' actually resides and where to put it. If you can guide me a bit more that would be really helpful. Thank you. |
@Akash-Kumar-Sen let's take a gitlab importer for example https://github.com/nexB/vulnerablecode/blob/main/vulnerabilities/importers/gitlab.py. We go to the repository ( data source ) https://gitlab.com/gitlab-org/advisories-community/-/tree/main and search for a
|
@ziadhany so I have to do what you've just mentioned for the file https://github.com/nexB/vulnerablecode/blob/main/vulnerabilities/importers/github.py instead of https://github.com/nexB/vulnerablecode/blob/main/vulnerabilities/importers/gitlab.py? |
@Akash-Kumar-Sen So you need to go to github importer https://github.com/nexB/vulnerablecode/blob/main/vulnerabilities/importers/github.py . add this in line 165. |
@ziadhany thank you for your help, I am new to the world of open source, your help can be a milestone in my career. |
This is still open despite pull request #845. If @Akash-Kumar-Sen is not working on this right now, I can open a new pull request. |
Is the issue is still open and what is remaining to be done? |
https://github.com/github/advisory-database/blob/main/LICENSE.md use this as license url
The text was updated successfully, but these errors were encountered: