Skip to content

Workflows - #138

Merged
dorssel merged 2 commits into
mainfrom
workflows
Oct 17, 2026
Merged

dorssel merged 2 commits into
mainfrom
workflows

Conversation

@dorssel

@dorssel dorssel commented Oct 17, 2026

Copy link
Copy Markdown
Owner

Add release workflows

@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails
actions/actions/checkout 3d3c42e5aac5ba805825da76410c181273ba90b1 🟢 6.3
Details
CheckScoreReason
Code-Review🟢 10all changesets reviewed
Maintained🟢 45 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 4
Binary-Artifacts🟢 10no binaries found in the repo
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Pinned-Dependencies🟢 3dependency not pinned by hash detected -- score normalized to 3
Fuzzing⚠️ 0project is not fuzzed
Packaging⚠️ -1packaging workflow not detected
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Security-Policy🟢 9security policy file detected
SAST🟢 10SAST tool is run on all commits
Branch-Protection🟢 6branch protection is not maximal on development and all release branches
actions/slsa-framework/slsa-github-generator/.github/workflows/generator_generic_slsa3.yml f7dd8c54c2067bafc12ca7a55595d5ee9b75204a 🟢 6.7
Details
CheckScoreReason
Maintained⚠️ 11 commit(s) and 1 issue activity found in the last 90 days -- score normalized to 1
Code-Review🟢 10all changesets reviewed
Dependency-Update-Tool🟢 10update tool detected
Security-Policy🟢 10security policy file detected
Binary-Artifacts🟢 10no binaries found in the repo
Packaging⚠️ -1packaging workflow not detected
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Token-Permissions🟢 10GitHub workflow tokens follow principle of least privilege
Pinned-Dependencies🟢 4dependency not pinned by hash detected -- score normalized to 4
CII-Best-Practices🟢 5badge detected: Passing
Signed-Releases🟢 105 out of the last 5 releases have a total of 5 signed artifacts.
License🟢 10license file detected
SAST🟢 7SAST tool detected but not run on all commits
Fuzzing⚠️ 0project is not fuzzed
Branch-Protection⚠️ 2branch protection is not maximal on development and all release branches
Vulnerabilities⚠️ 0239 existing vulnerabilities detected
CI-Tests⚠️ 02 out of 28 merged PRs checked by a CI test -- score normalized to 0
Contributors🟢 10project has 33 contributing companies or organizations

Scanned Files

  • .github/workflows/slsa.yml

@codecov

codecov Bot commented Oct 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (e0a0db8) to head (b9e0c50).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #138   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines           37        37           
  Branches         3         3           
=========================================
  Hits            37        37           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

Both workflows contain critical blockers that prevent successful execution.

2 open issues
What changed in this PR

Adds GitHub Actions workflows for NuGet publishing and SLSA provenance generation.

Changes:

  • Publishes release packages to NuGet.
  • Generates and uploads provenance for release assets.
  • Supports release-triggered and manual workflows.
File Summary Findings
.github/​workflows/​nuget.yml Publishes release packages to NuGet. Critical: unsupported case() expression prevents execution (4 votes).
.github/​workflows/​slsa.yml Generates and uploads SLSA provenance. Critical: the pinned SHA cannot use the default builder path (4 votes).

Review effort: Lite

Comment thread .github/workflows/nuget.yml
Comment thread .github/workflows/slsa.yml
@github-actions

Copy link
Copy Markdown
Contributor

✅MegaLinter analysis: Success

Descriptor Linter Files Fixed Errors Max errors Warnings Elapsed time
✅ ACTION actionlint 8 0 0 0.09s
✅ ACTION zizmor 8 0 0 2.98s
✅ EDITORCONFIG editorconfig-checker 48 0 0 0.04s
✅ JSON jsonlint 2 0 0 0.08s
✅ JSON prettier 2 0 0 0.33s
✅ JSON v8r 2 0 0 3.96s
✅ MARKDOWN markdownlint 2 0 0 0.53s
✅ MARKDOWN markdown-table-formatter 2 0 0 0.23s
✅ REPOSITORY betterleaks yes no no 0.71s
✅ REPOSITORY checkov yes no no 22.54s
✅ REPOSITORY git_diff yes no no 0.01s
✅ REPOSITORY grype yes no no 113.11s
✅ REPOSITORY osv-scanner yes no no 0.54s
✅ REPOSITORY secretlint yes no no 1.05s
✅ REPOSITORY syft yes no no 2.53s
✅ REPOSITORY trivy-sbom yes no no 0.66s
✅ REPOSITORY trufflehog yes no no 5.2s
✅ XML xmllint 14 0 0 0.14s
✅ YAML prettier 13 0 0 0.71s
✅ YAML v8r 13 0 0 12.76s
✅ YAML yamllint 13 0 0 0.66s

See detailed reports in MegaLinter artifacts

MegaLinter is provided by OX Security
Show us your support by starring ⭐ the repository

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The NuGet workflow does not trigger for normal published releases.

1 open issue
2 resolved since last review

Review effort: Lite

Comment thread .github/workflows/nuget.yml

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

Release and provenance workflows require final human review.

No open issues 🎉

1 resolved since last review

Review effort: Lite

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approved

No unresolved blocking issues remain after accounting for the intentional SLSA branch workaround and developer clarifications.

No open issues 🎉

Review effort: Balanced

@dorssel
dorssel merged commit e578ab3 into main Oct 17, 2026
13 checks passed
@dorssel
dorssel deleted the workflows branch October 17, 2026 08:45
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