Skip to content

Conversation

@SamMorrowDrums
Copy link
Collaborator

Summary

Rebased version of #1875 by @alexandear - preserving original authorship.

Removes the unused github.com/fatih/color dependency and adds go mod tidy -diff to CI for better dependency validation.

Changes

  • Removed github.com/fatih/color from go.mod and go.sum
  • Cleaned up transitive dependencies (mattn/go-colorable, mattn/go-isatty)
  • Changed CI workflow from go mod download to go mod tidy -diff
  • Added Windows CRLF workaround for go mod tidy -diff

Closes #1875

Co-authored-by: Oleksandr Redko oleksandr.red+github@gmail.com

@SamMorrowDrums SamMorrowDrums requested a review from a team as a code owner January 26, 2026 14:22
Copilot AI review requested due to automatic review settings January 26, 2026 14:22
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 PR removes the unused github.com/fatih/color dependency and improves CI dependency validation by switching from go mod download to go mod tidy -diff.

Changes:

  • Removed unused github.com/fatih/color dependency and its transitive dependencies (mattn/go-colorable, mattn/go-isatty)
  • Changed CI workflow from go mod download to go mod tidy -diff for automatic validation of dependency cleanliness
  • Added Windows-specific workaround for CRLF line ending issues that would cause go mod tidy -diff to fail

Reviewed changes

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

File Description
go.mod Removed github.com/fatih/color from direct dependencies
go.sum Removed checksums for fatih/color and its transitive dependencies (mattn/go-colorable, mattn/go-isatty, and unused golang.org/x/sys versions)
.github/workflows/go.yml Added Windows CRLF workaround and changed from go mod download to go mod tidy -diff for better CI validation

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.

4 participants