Skip to content

Add support for go fix in CI - #14

Merged
navaneeth-dev merged 1 commit into
mainfrom
ci/go-fix-check
Oct 26, 2026
Merged

navaneeth-dev merged 1 commit into
mainfrom
ci/go-fix-check

Conversation

@navaneeth-dev

@navaneeth-dev navaneeth-dev commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

This PR adds support for go fix in CI.

Go fix uses a suite of algorithms to identify opportunities to improve your code, often by taking advantage of more modern features of the language and library.

This can be fast-tracked and merged first. @elmiko

Add a CI step that runs go fix -diff, which fails when go fix would
rewrite any code. Apply the one fix it currently suggests on main
(strings.SplitSeq in users.go).

Signed-off-by: Navaneeth <me@rizexor.com>
@navaneeth-dev
navaneeth-dev requested a review from a team as a code owner October 22, 2026 14:27
Copilot AI balanced review requested due to automatic review settings October 22, 2026 14:27

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@navaneeth-dev navaneeth-dev changed the title ci: check that go fix has no pending changes Add support for go fix in CI Oct 22, 2026
@navaneeth-dev navaneeth-dev self-assigned this Oct 22, 2026
@navaneeth-dev
navaneeth-dev requested a review from elmiko October 22, 2026 14:30

@elmiko elmiko left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@elmiko

elmiko commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator

will probably need a review from @t-lo or @tormath1

@tormath1 tormath1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@navaneeth-dev
navaneeth-dev merged commit ed55286 into main Oct 26, 2026
6 checks passed
@navaneeth-dev
navaneeth-dev deleted the ci/go-fix-check branch October 26, 2026 11:38
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