Skip to content

Add Snapshot inputs #118

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

Merged
merged 3 commits into from
Jun 16, 2025
Merged

Add Snapshot inputs #118

merged 3 commits into from
Jun 16, 2025

Conversation

ljones140
Copy link
Contributor

Adding inputs required for DG autosubmission.

New Inputs:

  • snapshot-sha
  • snapshot-ref
  • detector-name
  • detector-version
  • detector-url

All are optional.
If any Detector inputs are provided, all three must be provided.

Note

To keep consistency with the other actions I have uses - in the names snapshot-sha. For example see Go Action PR.

But this action uses camel case for other inputs such as detectorsFilters and detectorArgs
Happy to change if an issue, but thought we should keep consistency with the other actions.

Testing

Testing on a forked C# repo https://github.com/ljones140/vs-streamjsonrpc

When passing inputs:

{
  "detector": {
    "name": "Lewis Detector",
    "url": "http://www.example.com",
    "version": "1.1.1"
  },
....
  "sha": "53ebfbe5235cbb88a8edfdf55b57f2f1fae8df9d",
  "ref": "refs/heads/main"
}

When only one detector passed we receive error:
Screenshot 2025-06-16 at 11 44 49

Part of: https://github.com/github/dependency-graph/issues/6451

Optional but if any are provided, then all are required
@ljones140 ljones140 requested review from a team as code owners June 16, 2025 10:54
@ljones140 ljones140 requested review from felickz and adrienpessu and removed request for a team June 16, 2025 10:54
@ljones140 ljones140 merged commit 04aaaf6 into main Jun 16, 2025
5 checks passed
@ljones140 ljones140 deleted the ljones140/add-snapshot-inputs branch June 16, 2025 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants