-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Broken version
and --version
with v1.32.0
#1468
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
Comments
Hey, thank you for opening your first Issue ! 🙂 If you would like to contribute we have a guide for contributors. |
@ybiquitous thanks for reporting this one, seems like something is broken due to recent refactor docker build for multi-arch support. Do you plan to work on this? I can take a look sometimes in weekend only. |
@sayboras Sorry, I'm not familiar with the Go language... 🙇 |
no worry, and thanks again for reporting this :) |
Same error here from golangci/golangci-lint:latest docker image ^^ |
As part of golangci#1383, multi-arch docker build was supported. However, ldflags for version details was missing. This commit is to add -ldflags as part of Docker build. Fixes golangci#1468 Signed-off-by: Tam Mach <[email protected]>
As part of golangci#1383, multi-arch docker build was supported. However, ldflags for version details was missing. This commit is to add -ldflags as part of Docker build. Fixes golangci#1468 Signed-off-by: Tam Mach <[email protected]>
As part of golangci#1383, multi-arch docker build was supported. However, ldflags for version details was missing. This commit is to add -ldflags as part of Docker build. Fixes golangci#1468 Signed-off-by: Tam Mach <[email protected]>
As part of golangci#1383, multi-arch docker build was supported. However, ldflags for version details was missing. This commit is to add -ldflags as part of Docker build. Fixes golangci#1468 Signed-off-by: Tam Mach <[email protected]>
As part of golangci#1383, multi-arch docker build was supported. However, ldflags for version details was missing. This commit is to add -ldflags as part of Docker build. Fixes golangci#1468 Signed-off-by: Tam Mach <[email protected]>
@nvuillam thanks, latest tag is pointing to v1.32.0, hence it will face the same issue. I got sometime today, so just send the fix. It's pending for review, and might take sometimes. |
* build(docker): Fix version details in docker image As part of #1383, multi-arch docker build was supported. However, ldflags for version details was missing. This commit is to add -ldflags as part of Docker build. I take this chance to refactor github action as well. Fixes #1468 Signed-off-by: Tam Mach <[email protected]>
Thank you so much for the quick fix! 😊 |
Version 1.32.1 was released just now, I have performed quick check, seems fixed now
|
It works for me too, thanks :) |
It works for me. Thank you so much! 😊 |
Thank you for creating the issue!
Please include the following information:
Version of golangci-lint
v1.32.0
Config file
None.
Go environment
None. (Docker)
Verbose output of running
v1.31.0:
v1.32.0:
I noticed that
golangci-lint version
andgolangci-lint --version
are broken with v1.32.0:version
outputs invalid text:golangci-lint has version master built from ? on
--version
is handled as an unknown flag:Error: unknown flag: --version
The text was updated successfully, but these errors were encountered: