-
Notifications
You must be signed in to change notification settings - Fork 24
Pin actions/checkout to commit hash with version comment #376
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pin actions/checkout to commit hash with version comment #376
Conversation
Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
.github/workflows/cpp-linter.yml
Outdated
| runs-on: ubuntu-latest | ||
| steps: | ||
| # v6.0.0 | ||
| - uses: actions/checkout@v6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot please add comments after - uses: actions/checkout@v6, for example - uses: actions/checkout@v6 # v6.0.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in ce0a7c7. Changed all version comments to inline format:
- uses: actions/checkout@v6 # v6.0.0Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com>
|
@copilot do not update for |
Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com>
.github/workflows/cpp-linter.yml
Outdated
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v6 | ||
| - uses: actions/checkout@v6 # 1af3b93b6815bc44a9784bd300feb67ff0d1eeb3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot I expect to update to the below
- uses: actions/checkout@1af3b93b6815bc44a9784bd300feb67ff0d1eeb3 # v6.0.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in 41a3b15. Updated to use commit hash as the reference with the semantic version tag as the comment:
- uses: actions/checkout@1af3b93b6815bc44a9784bd300feb67ff0d1eeb3 # v6.0.0Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com>
3a7f15e
into
dependabot/github_actions/actions-76468cb07f
…375) * chore(deps): bump actions/checkout from 5 to 6 in the actions group Bumps the actions group with 1 update: [actions/checkout](https://github.com/actions/checkout). Updates `actions/checkout` from 5 to 6 - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v5...v6) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major dependency-group: actions ... Signed-off-by: dependabot[bot] <support@github.com> * Pin actions/checkout to commit hash with version comment (#376) * Initial plan * Add version comments to actions/checkout references Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com> * Move version comments to inline format Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com> * Revert examples folder changes and use commit hash in comments Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com> * Use commit hash as reference and version tag as comment Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com> * do not persist git credentials when not needed and make explicit when we do need it --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: shenxianpeng <3353385+shenxianpeng@users.noreply.github.com> Co-authored-by: Brendan <2bndy5@gmail.com>
Updated
actions/checkoutreferences in workflow files to use the full commit hash instead of the version tag, with the semantic version as an inline comment for clarity.Changes
actions/checkoutto use commit hash1af3b93b6815bc44a9784bd300feb67ff0d1eeb3with# v6.0.0comment in:.github/workflows/release.yml.github/workflows/self-test.yml.github/workflows/cpp-linter.ymlExample
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.