Skip to content

Use SHA to reference GitHub Actions#227

Merged
mmoll merged 4 commits into
mayflower:masterfrom
xalopp:gha_pinact
Dec 9, 2025
Merged

Use SHA to reference GitHub Actions#227
mmoll merged 4 commits into
mayflower:masterfrom
xalopp:gha_pinact

Conversation

@xalopp

@xalopp xalopp commented Dec 5, 2025

Copy link
Copy Markdown
Collaborator

Type of PR

  • Bugfix
  • New Feature
  • Other (explain):

Fixes Issues reported by SonarQube:
https://docs.github.com/en/actions/reference/security/secure-use#using-third-party-actions

Breaking changes

  • Yes, this is a breaking change

Beside simply using the SHA for the versions, this also updates some of the GitHub Actions

Description

I use https://github.com/suzuki-shunsuke/pinact for is PR.

Comment thread .github/workflows/ci.yml Outdated
@xalopp

xalopp commented Dec 5, 2025

Copy link
Copy Markdown
Collaborator Author

@mmoll please 👀 this PR

@xalopp xalopp mentioned this pull request Dec 5, 2025
4 tasks
@mmoll mmoll closed this Dec 8, 2025
@mmoll mmoll reopened this Dec 8, 2025
@mmoll mmoll closed this Dec 8, 2025
@mmoll mmoll reopened this Dec 8, 2025
@codecov

codecov Bot commented Dec 8, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (93dad05) to head (d1a353b).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##              master      #227   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity       147       147           
===========================================
  Files              8         8           
  Lines            669       669           
===========================================
  Hits             669       669           

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

@sonarqubecloud

sonarqubecloud Bot commented Dec 9, 2025

Copy link
Copy Markdown

Comment thread .github/workflows/ci.yml
@mmoll mmoll merged commit c2f1f05 into mayflower:master Dec 9, 2025
16 checks passed
@mmoll

mmoll commented Dec 9, 2025

Copy link
Copy Markdown
Contributor

merged, merci @xalopp!

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