Skip to content

Comments

CLOUDP-347875: Move GCI to go tool instead of Devbox#2752

Merged
roothorp merged 2 commits intomainfrom
cloudp-347875-devbox-gci-dep
Oct 1, 2025
Merged

CLOUDP-347875: Move GCI to go tool instead of Devbox#2752
roothorp merged 2 commits intomainfrom
cloudp-347875-devbox-gci-dep

Conversation

@roothorp
Copy link
Collaborator

@roothorp roothorp commented Sep 30, 2025

Summary

GCI was causing issues with Devbox updates, so we can instead install it as a go tool and remove it from Devbox to circumvent the issue.

This does mean that we must invoke gci with go tool - the only place I found we use gci was in the fmt make target, which I have updated.

Proof of Work

Checklist

  • Have you linked a jira ticket and/or is the ticket in the title?
  • Have you checked whether your jira ticket required DOCSP changes?
  • Have you checked for release_note changes?
  • Have you signed our CLA?

Reminder (Please remove this when merging)

  • Please try to Approve or Reject Changes the PR, keep PRs in review as short as possible
  • Remember the following Communication Standards - use comment prefixes for clarity:
    • blocking: Must be addressed before approval.
    • follow-up: Can be addressed in a later PR or ticket.
    • q: Clarifying question.
    • nit: Non-blocking suggestions.
    • note: Side-note, non-actionable. Example: Praise
    • --> no prefix is considered a question

@roothorp roothorp requested a review from a team as a code owner September 30, 2025 10:23
@roothorp roothorp merged commit c022b1b into main Oct 1, 2025
22 checks passed
@roothorp roothorp deleted the cloudp-347875-devbox-gci-dep branch October 1, 2025 15:18
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