Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(cmd/gazelle): small cleanups to fix-update #2017

Merged
merged 1 commit into from
Jan 22, 2025

Conversation

tyler-french
Copy link
Contributor

@tyler-french tyler-french commented Jan 22, 2025

There are some small changes to fix update:

  • remove some legacy code that we don't need anymore
  • error handling simplification
  • unnecessary loops

cmd/gazelle/fix-update.go Show resolved Hide resolved
cmd/gazelle/fix-update.go Show resolved Hide resolved
@tyler-french tyler-french changed the title small cleanups to fix-update fix(cmd/gazelle): small cleanups to fix-update Jan 22, 2025
@tyler-french tyler-french marked this pull request as ready for review January 22, 2025 16:04
@tyler-french tyler-french merged commit 1bbedea into bazel-contrib:master Jan 22, 2025
14 checks passed
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.

None yet

2 participants