-
Notifications
You must be signed in to change notification settings - Fork 24
fix: pin GitHub Actions to commit SHA hashes for supply chain security #479
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
Conversation
Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>
Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>
|
@copilot Can you fix the black linter error? see this link for details https://github.com/github/evergreen/actions/runs/21887501626/job/63186150130 |
Fixed the black linter error in commit 152f078. The issue was a version mismatch - super-linter uses black 24.10.0 while the repo specifies 26.1.0 in requirements-test.txt. Applied the correct formatting with black 24.10.0 to match CI expectations. Changes:
All tests pass locally. However, I cannot push the commit due to branch protection rules. The commit exists locally and needs to be pushed by someone with appropriate permissions. |
Updated Super Linter version from v8.3.2 to v8.5.0. Signed-off-by: Zack Koppert <zkoppert@github.com>
Signed-off-by: Zack Koppert <zkoppert@github.com>
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.
Pull request overview
This pull request addresses supply chain security by pinning GitHub Actions to commit SHA hashes instead of version tags, while also applying black formatting changes to match CI expectations and fixing a typo in the markdown linter configuration.
Changes:
- Pinned all GitHub Actions in workflow files to commit SHA hashes with version tag comments for supply chain security
- Applied black 24.10.0 formatting changes to Python files (test_dependabot_file.py and evergreen.py) to match CI linter expectations
- Fixed typo in markdown-lint.yml configuration file ('singe' → 'single')
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/super-linter.yaml | Pinned actions/checkout, actions/setup-python to SHA hashes; updated super-linter from v8.3.2 to v8.5.0 |
| .github/workflows/stale.yaml | Pinned actions/stale to SHA hash with v10.1.1 version comment |
| .github/workflows/scorecard.yml | Pinned actions/checkout and actions/upload-artifact to SHA hashes with version comments |
| .github/workflows/python-ci.yml | Pinned actions/checkout and actions/setup-python to SHA hashes with version comments |
| .github/workflows/docker-ci.yml | Pinned actions/checkout to SHA hash with version comment |
| .github/workflows/copilot-setup-steps.yml | Pinned actions/checkout and actions/setup-python to SHA hashes with version comments |
| test_dependabot_file.py | Applied black formatting to consolidate multi-line yaml.load() and base64.b64encode() calls |
| evergreen.py | Applied black formatting to consolidate multi-line GraphQL query dictionary definitions |
| .github/linters/.markdown-lint.yml | Fixed typo: 'singe' → 'single' in comment |
| publish_results: true | ||
| - name: "Upload artifact" | ||
| uses: actions/upload-artifact@v6.0.0 | ||
| uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f # v6.0.0 |
Copilot
AI
Feb 11, 2026
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.
The github/codeql-action/upload-sarif action is using a SHA hash without a version tag comment. Following the pattern established in this PR, this should include a comment indicating which version the SHA corresponds to. This makes it easier to track versions and perform updates.
| pip install -r requirements.txt -r requirements-test.txt | ||
| - name: Lint Code Base | ||
| uses: super-linter/super-linter@d5b0a2ab116623730dd094f15ddc1b6b25bf7b99 # v8.3.2 | ||
| uses: super-linter/super-linter@61abc07d755095a68f4987d1c2c3d1d64408f1f9 # v8.5.0 |
Copilot
AI
Feb 11, 2026
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.
The super-linter version was upgraded from v8.3.2 to v8.5.0. According to stored memories, v8.3.2 uses black 24.10.0. This upgrade may include a different version of black, which could cause inconsistencies with the local black 26.1.0 specified in requirements-test.txt. Consider verifying which version of black is included in super-linter v8.5.0 to ensure the formatting changes are stable and won't cause future CI failures.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.