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.

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

feat: Implement dependency review and dependency submission APIs #2932

Merged
merged 1 commit into from
Jun 14, 2024

Conversation

awedist
Copy link
Contributor

@awedist awedist commented Jun 13, 2024

Relates to #2825


Before the change?

Octokit did not support the Dependency Graph APIs.

After the change?

  • Ability to request dependency diffs between two commits.
  • Ability to create a dependency snapshot for a given repository.
  • Ability to request a Software Bill of Materials for a given repository.

I could not implement the SBOM functionality because the SBOM GitHub API provides a JSON where the keys are in camel case.
If there is a way to configure the deserializer to accept camel case, please let me know.

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@nickfloyd
Copy link
Contributor

Hey @awedist Thank you for the changes here ❤️ . The SPDX format and the rigidity of how serialization has been implemented will make things a bit more challenging here. We'll need to look to see if the model properties can be annotated to cover the SBOM APIs - I'm pretty sure we could implement a custom naming strategy for the serializer to be able to use or add serializer options. We'd most likely have to modify the implementation in SimpleJson.cs.

I'll approve and merge this in as is and we can revisit the other API once we've taken some time to look.

@nickfloyd nickfloyd changed the title Implement dependency review and dependency submission APIs feat: Implement dependency review and dependency submission APIs Jun 14, 2024
@nickfloyd nickfloyd added the Type: Feature New feature or request label Jun 14, 2024
@nickfloyd nickfloyd merged commit 7d54cb0 into octokit:main Jun 14, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New feature or request
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants