Skip to content

[FIX] Update dependencies - #1185

Merged
bbengfort merged 1 commit into
mainfrom
modernize
Apr 12, 2026
Merged

[FIX] Update dependencies#1185
bbengfort merged 1 commit into
mainfrom
modernize

Conversation

@bbengfort

@bbengfort bbengfort commented Apr 12, 2026

Copy link
Copy Markdown
Collaborator

Scope of changes

Updates dependencies and appeases linter and tests.

Type of change

  • bug fix
  • new feature
  • documentation
  • other (describe)

Acceptance criteria

This PR will be merged without review.

Author checklist

  • I have manually tested the change and/or added automation in the form of unit tests or integration tests
  • I have updated the dependencies list
  • I have recompiled and included new protocol buffers to reflect changes I made
  • I have added new test fixtures as needed to support added tests

Note

Medium Risk
Medium risk due to the Go toolchain jump to 1.26 and broad dependency upgrades, which can introduce subtle build/runtime changes and regenerated protobuf output. Workflow action version bumps are low risk but could affect CI/container builds if compatibility issues arise.

Overview
Upgrades the codebase to Go 1.26.x (module go 1.26.1, CI, and container builder images) and bumps staticcheck plus a broad set of Go module dependencies (e.g. gin, grpc, google.golang.org/api, sentry, auth0, prometheus, trisa).

Regenerates protobuf outputs (pkg/bff/models/v1/models.pb.go, pkg/gds/admin/v1/*.pb.go) with newer protoc/generators (notably switching raw descriptor handling to unsafe string slicing), updates GitHub Actions checkouts to actions/checkout@v6, and adjusts admin tests to expect the updated review-note author email (admin@trisa.io).

Reviewed by Cursor Bugbot for commit 15f6d74. Configure here.

Comment thread pkg/gds/admin_test.go

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This email addresses are in the fixtures … since we're going to be doing GDS 2.0 soon I figured there wasn't a point in changing the fixtures which is actually a lot of work.

@bbengfort
bbengfort merged commit d727bf6 into main Apr 12, 2026
15 of 16 checks passed
@bbengfort
bbengfort deleted the modernize branch April 12, 2026 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant