Skip to content

Fix makefile goimports target#75

Merged
hardikdr merged 2 commits intomainfrom
fix/fix-fmt-make-target
Jun 28, 2024
Merged

Fix makefile goimports target#75
hardikdr merged 2 commits intomainfrom
fix/fix-fmt-make-target

Conversation

@damyan
Copy link
Copy Markdown
Member

@damyan damyan commented Jun 28, 2024

Missing GOIMPORTS and GOIMPORTS_VERSION variables leads to a silently failing goimport target, as well as all its reverse dependency targets. Example:

make fmt

w .
12:01  up 1 day, 40 mins, 6 users, load averages: 4.14 2.66 2.51
USER       TTY      FROM    LOGIN@  IDLE WHAT```

Missing `GOIMPORTS` and `GOIMPORTS_VERSION` variables leads to a
silently failing goimport target, as well as all its reverse dependency
targets. Example:
```
make fmt
w .
11:58  up 1 day, 37 mins, 6 users, load averages: 1.40 1.74 2.26
USER       TTY      FROM    LOGIN@  IDLE WHAT```
@damyan damyan requested a review from hardikdr June 28, 2024 10:02
@damyan damyan enabled auto-merge (squash) June 28, 2024 10:08
@hardikdr hardikdr disabled auto-merge June 28, 2024 13:16
@hardikdr hardikdr merged commit 9a07bbb into main Jun 28, 2024
@hardikdr hardikdr deleted the fix/fix-fmt-make-target branch June 28, 2024 13:17
@afritzler afritzler added the chore label Jul 2, 2024
@hardikdr hardikdr added this to Roadmap Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants