Skip to content

go.mod: Bump scorecard/v5 to v5.4.1-0.20260416042454-4b8e9d1b5db8#817

Merged
jeffmendoza merged 2 commits intoossf:mainfrom
justaugustus:bump-scorecard
Apr 17, 2026
Merged

go.mod: Bump scorecard/v5 to v5.4.1-0.20260416042454-4b8e9d1b5db8#817
jeffmendoza merged 2 commits intoossf:mainfrom
justaugustus:bump-scorecard

Conversation

@justaugustus
Copy link
Copy Markdown
Member

@justaugustus justaugustus commented Apr 16, 2026

Scorecard is prepping a release, but we still have some tweaks before it's ready to go.
In the meantime, let's capture the dep bumps from https://github.com/ossf/scorecard/tree/4b8e9d1b5db8f986de4f4c666f05694f8252bfa4.

Signed-off-by: Stephen Augustus <foo@auggie.dev>
@justaugustus justaugustus requested a review from a team as a code owner April 16, 2026 05:24
@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Apr 16, 2026
@justaugustus
Copy link
Copy Markdown
Member Author

@copilot — Fix failing status checks

Copy link
Copy Markdown
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

Updates the project’s Go dependencies to track an unreleased commit of github.com/ossf/scorecard/v5 as part of Scorecard’s release preparation.

Changes:

  • Bump github.com/ossf/scorecard/v5 to v5.4.1-0.20260416042454-4b8e9d1b5db8.
  • Refresh the indirect dependency set in go.mod to align with the new Scorecard commit.
  • Regenerate go.sum entries to match the updated module graph.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
go.mod Bumps Scorecard and updates/introduces a number of indirect requirements pulled in by the new version.
go.sum Updates checksum set to reflect the new resolved dependency graph.

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

Comment thread go.mod
- Run go mod tidy to remove dgrijalva/jwt-go (archived/unmaintained)
  and 10 other stale indirect dependencies no longer in the module graph
- Extract "open" and "closed" string literals in pkg/issue/issue.go into
  issueStateOpen/issueStateClosed constants to fix goconst lint failures

Addresses PR ossf#817 review feedback:
- Copilot flagged dgrijalva/jwt-go supply-chain risk
- CI lint job failed on goconst violations

Signed-off-by: Stephen Augustus <foo@auggie.dev>
Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Stephen Augustus <foo@auggie.dev>
Copy link
Copy Markdown
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

Copilot reviewed 2 out of 3 changed files in this pull request and generated no new comments.


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

@jeffmendoza jeffmendoza merged commit 53e291f into ossf:main Apr 17, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants