Skip to content
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.

Sign up for GitHub

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

Jump to bottom

add DepsDataSource #789

Merged
merged 4 commits into from
Nov 18, 2022
Merged

add DepsDataSource #789

merged 4 commits into from
Nov 18, 2022

Conversation

keshav-space
Copy link
Member

@keshav-space keshav-space commented Jul 5, 2022 •
edited
Loading

  • add Deps DataSource
  • add tests for Deps DataSource

@keshav-space keshav-space marked this pull request as draft July 5, 2022 09:51
@keshav-space keshav-space added the VulnTotal Tool for cross-validating vulnerability label Jul 5, 2022
@keshav-space keshav-space self-assigned this Jul 5, 2022
@keshav-space keshav-space changed the title Deps validator and CLI support add Deps validator Jul 19, 2022
@keshav-space keshav-space marked this pull request as ready for review July 19, 2022 20:45
@keshav-space keshav-space force-pushed the deps_validator branch 2 times, most recently from 6a06a82 to 0dc4ca9 Compare July 25, 2022 13:27
@keshav-space keshav-space changed the title add Deps validator add DepsDataSource Aug 3, 2022
@keshav-space keshav-space force-pushed the deps_validator branch 2 times, most recently from ec71558 to 3c51d9d Compare August 5, 2022 12:34
@keshav-space keshav-space marked this pull request as draft August 25, 2022 20:03
@keshav-space keshav-space force-pushed the deps_validator branch 2 times, most recently from 3c51d9d to 741838b Compare August 26, 2022 18:51
@keshav-space keshav-space marked this pull request as ready for review August 26, 2022 18:53
@keshav-space
add Deps Datasource
e30131f 
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
@keshav-space
add tests for Deps Datasource
3235316 
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
@TG1999
Copy link
Contributor

TG1999 commented Sep 5, 2022

Add doctsrings for all the functions and also add doctests/ unit tests for functions which have not been tested at all

@TG1999 TG1999 added this to the v31.0 milestone Oct 11, 2022
@keshav-space keshav-space mentioned this pull request Oct 12, 2022
Closed
9 tasks
@pombredanne
Use standar header
8410076 
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@pombredanne
Use standard header
5cdf24a 
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
pombredanne
pombredanne approved these changes Nov 18, 2022
View reviewed changes
Copy link
Member

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

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

LGTM .... Merging! 👍

@pombredanne pombredanne merged commit ab10ca0 into aboutcode-org:vulntotal Nov 18, 2022
@keshav-space keshav-space deleted the deps_validator branch January 10, 2023 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pombredanne pombredanne pombredanne approved these changes

@Hritik14 Hritik14 Awaiting requested review from Hritik14

@sbs2001 sbs2001 Awaiting requested review from sbs2001

@TG1999 TG1999 Awaiting requested review from TG1999

Assignees

@keshav-space keshav-space

Labels
VulnTotal Tool for cross-validating vulnerability
Projects
None yet
Milestone
v31.0
Development

Successfully merging this pull request may close these issues.
3 participants
@keshav-space @TG1999 @pombredanne