Skip to content

ci(github-actions): implement dependency review workflow - #9

Merged
karlpip merged 1 commit into
mainfrom
ci/add-license-check
Sep 30, 2026
Merged

karlpip merged 1 commit into
mainfrom
ci/add-license-check

Conversation

@ogruene

@ogruene ogruene commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Add Dependency Review Workflow

Automates license compliance checks on every PR targeting main, enforcing Eclipse Foundation license policy.

What it does

  • Triggers on every pull request targeting main
  • Fetches approved and valid licenses dynamically from the Eclipse Foundation license API — always up to date, no hardcoded list
  • Merges project-specific license allowances from .github/additional-allowed-licenses.txt
  • Runs actions/dependency-review-action@v5 pinned to a specific commit SHA for supply chain safety
  • Posts a license and vulnerability summary comment directly on the PR
  • Shows available patched versions for any vulnerable dependency

Additional allowed licenses

License Reason
LicenseRef-scancode-google-patent-license-golang Required for Go standard library dependencies

Permissions

Runs with minimal permissions: contents: read, pull-requests: write (needed for PR comment).

@github-actions

Copy link
Copy Markdown

Dependency Review

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

OpenSSF Scorecard

PackageVersionScoreDetails
actions/actions/checkout 3d3c42e5aac5ba805825da76410c181273ba90b1 🟢 6.4
Details
CheckScoreReason
Code-Review🟢 10all changesets reviewed
Maintained🟢 57 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 5
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
License🟢 10license file detected
Fuzzing⚠️ 0project is not fuzzed
Packaging⚠️ -1packaging workflow not 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/actions/dependency-review-action a1d282b36b6f3519aa1f3fc636f609c47dddb294 🟢 7.7
Details
CheckScoreReason
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Security-Policy🟢 9security policy file detected
Packaging⚠️ -1packaging workflow not detected
Code-Review🟢 10all changesets reviewed
Maintained🟢 1028 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 10
Binary-Artifacts🟢 10no binaries found in the repo
Token-Permissions🟢 9detected GitHub workflow tokens with excessive permissions
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Pinned-Dependencies⚠️ 1dependency not pinned by hash detected -- score normalized to 1
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Branch-Protection🟢 6branch protection is not maximal on development and all release branches
SAST🟢 9SAST tool detected but not run on all commits

Scanned Files

  • .github/workflows/dependency-review.yml

@karlpip
karlpip merged commit 617a55d into main Sep 30, 2026
2 checks passed
@karlpip
karlpip deleted the ci/add-license-check branch September 30, 2026 07:10
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