Skip to content

Conversation

@tijmendj
Copy link
Contributor

@tijmendj tijmendj commented Jan 29, 2026

Part of https://github.com/channable/devops/issues/13319.

Some people argued that it would be more consistent if we also pinned GitHub owned (i.e. github.com/actions/checkout) Actions.

@tijmendj tijmendj requested review from Qqwy and Copilot January 29, 2026 15:50
Copy link

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 updates the GitHub Actions workflow to pin actions/checkout to a specific commit SHA instead of using the v4 tag reference, improving security and consistency by making all external Actions use commit SHAs.

Changes:

  • Replaced three instances of actions/checkout@v4 with the pinned commit SHA 0c366fd6a839edf440554fa01a7085ccba70ac98

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Qqwy
Copy link
Contributor

Qqwy commented Jan 30, 2026

Thank you Tijmen! I understand the sentiment, and while I think it is not the most important thing in the world, it definitely cannot hurt to pin GitHub's own actions as well.

@OpsBotPrime merge

@OpsBotPrime
Copy link
Contributor

Your merge request has been denied, because merging on Fridays is not recommended. To override this behaviour use the command merge on Friday.

@Qqwy
Copy link
Contributor

Qqwy commented Jan 30, 2026

@OpsBotPrime merge on Friday

@OpsBotPrime
Copy link
Contributor

Rebased as 4809da9, waiting for CI …

@OpsBotPrime
Copy link
Contributor

CI job 🟡 started.

@OpsBotPrime
Copy link
Contributor

The build failed ❌.

If this is the result of a flaky test, then tag me again with the retry command. Otherwise, push a new commit and tag me again.

@Qqwy
Copy link
Contributor

Qqwy commented Jan 30, 2026

@OpsBotPrime retry on Friday

@OpsBotPrime
Copy link
Contributor

Rebased as 4809da9, waiting for CI …

@OpsBotPrime
Copy link
Contributor

CI job 🟡 started.

@OpsBotPrime OpsBotPrime merged commit 4809da9 into master Jan 30, 2026
10 checks passed
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.

3 participants