-
Notifications
You must be signed in to change notification settings - Fork 1
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 docs for batch endpoint #169
add docs for batch endpoint #169
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perhaps i am the wrong person to review this... but i tried to be a user and nothing worked for me. otherwise, it looks good (not fond of adrmdr
, but that's me)
Thanks @janasheehan! Your name was randomly picked to review when I opened the PR, so please don't feel obligated to review it too closely! I'll request some others with more specific knowledge of the endpoint. Thanks again! |
…dpoint-and-update-single-package-endpoint-docs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
"riskScores": { | ||
"author": 1.0, | ||
"engineering": 1.0, | ||
"license": 1.0, | ||
"malicious_code": 0.01, | ||
"total": 0.05, | ||
"vulnerability": 1.0 | ||
} | ||
}, | ||
"version": "0.1.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe a nitpick, but maybe the example response can match same order specified?
name
: The name of the package.version
: The version of the package.ecosystem
: The ecosystem to which the package was published.code
: The HTTP status code.malware
: A boolean indicating if the package has been triaged as malware.riskScores
: An object containing risk scores for each of the 5 Phylum risk domains associated with the pa
…dpoint-and-update-single-package-endpoint-docs
Please hold off on merging/publishing this for now. |
Sounds good! Will convert to draft just to make sure it doesn't get merged accidentally. |
I am going to close this PR as we will not be including this text in the published docs. |
Adds docs for new
/batch
threat feed end point. Also adds docs about top level key additions to single package end point.Note that I will update the example in the docs with one that actually returns slightly more interesting results after this gets deployed and I can run it against prod instead of staging.