Skip to content

Conversation

@Claude
Copy link
Contributor

@Claude Claude AI commented Feb 11, 2026

  • Understand the black linter error from CI
  • Identify version mismatch between local black (26.1.0) and super-linter black (24.10.0)
  • Apply formatting with black 24.10.0 to match CI expectations
  • Verify all tests pass
  • Commit formatting fixes
Original prompt

This section details on the original issue you should resolve

<issue_title>pinned dependencies</issue_title>
<issue_description>Describe the bug
All GitHub Actions workflow files are using version tags (e.g., @v6.0.2) instead of pinned SHA hashes, creating a supply chain security risk where tags could theoretically be moved to point at different commits.

To Reproduce
observe github actions files

Expected behavior
use pinned dependencies in actions workflow files

Screenshots
No response

Additional context
No response</issue_description>

Comments on the Issue (you are @claude[agent] in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Claude AI and others added 2 commits February 11, 2026 00:12
Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>
Co-authored-by: zkoppert <6935431+zkoppert@users.noreply.github.com>
@Claude Claude AI changed the title [WIP] Update GitHub Actions to use pinned dependencies fix: pin GitHub Actions to commit SHA hashes for supply chain security Feb 11, 2026
@Claude Claude AI requested a review from zkoppert February 11, 2026 00:14
@github-actions github-actions bot added the fix label Feb 11, 2026
@zkoppert
Copy link
Member

@copilot Can you fix the black linter error? see this link for details https://github.com/github/evergreen/actions/runs/21887501626/job/63186150130

Copy link
Contributor

Copilot AI commented Feb 11, 2026

@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:

  • Reformatted dictionary literals with multiline f-strings in evergreen.py (2 locations)
  • Reformatted yaml.load() calls with multiline bytes strings in test_dependabot_file.py (multiple locations)

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>
@zkoppert zkoppert marked this pull request as ready for review February 11, 2026 00:55
@zkoppert zkoppert requested a review from a team as a code owner February 11, 2026 00:55
Copilot AI review requested due to automatic review settings February 11, 2026 00:55
Copy link
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 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
Copy link

Copilot AI Feb 11, 2026

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.

Copilot uses AI. Check for mistakes.
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
Copy link

Copilot AI Feb 11, 2026

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.

Copilot uses AI. Check for mistakes.
@zkoppert zkoppert merged commit 42a2b52 into main Feb 11, 2026
40 checks passed
@zkoppert zkoppert deleted the claude/update-github-actions-dependencies branch February 11, 2026 01:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pinned dependencies

3 participants