Skip to content

fix: update upload-sarif documentation#196

Merged
dotkas merged 1 commit intomasterfrom
dotkas/CLI-1110/fix-broken-sarif-upload-docs
Aug 26, 2025
Merged

fix: update upload-sarif documentation#196
dotkas merged 1 commit intomasterfrom
dotkas/CLI-1110/fix-broken-sarif-upload-docs

Conversation

@dotkas
Copy link
Copy Markdown
Contributor

@dotkas dotkas commented Aug 25, 2025

After the deprecation of v2 of the upload-sarif action, we need to update our documentation to reflect the same change.

@dotkas dotkas requested review from a team as code owners August 25, 2025 11:01
@snyk-io
Copy link
Copy Markdown

snyk-io bot commented Aug 25, 2025

🎉 Snyk checks have passed. No issues have been found so far.

code/snyk check is complete. No issues have been found. (View Details)

@dotkas dotkas force-pushed the dotkas/CLI-1110/fix-broken-sarif-upload-docs branch from 10a18be to 2566d9b Compare August 25, 2025 11:02
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This file is the only changed one, the rest is a side-effect from the generator script.

@dotkas dotkas enabled auto-merge (squash) August 25, 2025 11:11
@dotkas dotkas force-pushed the dotkas/CLI-1110/fix-broken-sarif-upload-docs branch from 2566d9b to 0f202f9 Compare August 26, 2025 13:41
@dotkas dotkas disabled auto-merge August 26, 2025 13:41
@dotkas dotkas enabled auto-merge (squash) August 26, 2025 13:41
@dotkas dotkas merged commit e222141 into master Aug 26, 2025
31 checks passed
@dotkas dotkas deleted the dotkas/CLI-1110/fix-broken-sarif-upload-docs branch August 26, 2025 13:42
detro added a commit to snyk/skemium that referenced this pull request Oct 28, 2025
* fix: restore ability of Snyk GH Action to work with new GH sarif

Related: snyk/actions#196
See: https://github.blog/changelog/2025-07-21-code-scanning-will-stop-combining-multiple-sarif-runs-uploaded-in-the-same-sarif-file/

* chore: bump versions

Closes #37
Closes #38

* feat: added README instructions on how to cut a new release

* fix: try using absolute path for sarif file

* chore: update `.tool-versions`

* fix: switch to `snyk` run locally (no docker image)
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