Skip to content

fix: improve OpenSSF Scorecard score#1446

Merged
liudger merged 1 commit intomainfrom
fix/scorecard-improvements
Apr 16, 2026
Merged

fix: improve OpenSSF Scorecard score#1446
liudger merged 1 commit intomainfrom
fix/scorecard-improvements

Conversation

@liudger
Copy link
Copy Markdown
Owner

@liudger liudger commented Apr 16, 2026

  • Use npm ci instead of npm install in linting workflow (Pinned-Dependencies)
  • Add SLSA build provenance attestation to release workflow (Signed-Releases)
  • Add attestations:write permission for provenance generation

- Use npm ci instead of npm install in linting workflow (Pinned-Dependencies)
- Add SLSA build provenance attestation to release workflow (Signed-Releases)
- Add attestations:write permission for provenance generation
Copilot AI review requested due to automatic review settings April 16, 2026 11:04
@liudger liudger added ci Work that improves the continue integration. security Marks a security issue that needs to be resolved asap. labels Apr 16, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the repository’s OpenSSF Scorecard results by tightening dependency installation practices and adding SLSA build provenance attestations to the release process.

Changes:

  • Switch the Prettier workflow from npm install to npm ci to ensure deterministic installs (Pinned-Dependencies).
  • Add GitHub build provenance attestation generation to the release workflow (Signed-Releases).
  • Grant the release job attestations: write permission to allow provenance publication.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/release.yaml Adds provenance attestation step and the required attestations: write job permission.
.github/workflows/linting.yaml Uses npm ci for reproducible Node dependency installation during Prettier runs.

@sonarqubecloud
Copy link
Copy Markdown

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.89%. Comparing base (881f3a5) to head (1877bd1).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1446   +/-   ##
=======================================
  Coverage   99.89%   99.89%           
=======================================
  Files           6        6           
  Lines         955      955           
  Branches      128      128           
=======================================
  Hits          954      954           
  Partials        1        1           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@liudger liudger merged commit 7ed0878 into main Apr 16, 2026
22 checks passed
@liudger liudger deleted the fix/scorecard-improvements branch April 16, 2026 12:56
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ci Work that improves the continue integration. security Marks a security issue that needs to be resolved asap.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants