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

golangci: Run install step with --help #97

Merged
merged 1 commit into from
May 14, 2024
Merged

golangci: Run install step with --help #97

merged 1 commit into from
May 14, 2024

Conversation

r-hang
Copy link
Contributor

@r-hang r-hang commented May 14, 2024

PR builds have been failing with an unknown flag error

run golangci-lint
  Running [/home/runner/golangci-lint-1.58.1-linux-amd64/golangci-lint run  --version] in [/home/runner/work/cff/cff] ...
  Error: unknown flag: --version
  Failed executing command with error: unknown flag: --version

ref: https://github.com/uber-go/cff/actions/runs/9061971983/job/24894837626?pr=96

--version is a valid flag for golangci-lint but not golangci-lint run.

For golangci-lint run, the closest no-op action is --help.

Fix golangci-lint runs by using the --help flag instead of the version flag.

PR builds have been failing with an unknown flag error
```
run golangci-lint
  Running [/home/runner/golangci-lint-1.58.1-linux-amd64/golangci-lint run  --version] in [/home/runner/work/cff/cff] ...
  Error: unknown flag: --version
  Failed executing command with error: unknown flag: --version
```
ref: https://github.com/uber-go/cff/actions/runs/9061971983/job/24894837626?pr=96

--version is a valid flag for golangci-lint but not golangci-lint run.

For golangci-lint run, the closest no-op action is --help.

Fix golangci-lint runs by using the --help flag instead of the version
flag.
Copy link

codecov bot commented May 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.94%. Comparing base (e6aa6c5) to head (67e3583).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #97   +/-   ##
=======================================
  Coverage   67.94%   67.94%           
=======================================
  Files          32       32           
  Lines        2059     2059           
=======================================
  Hits         1399     1399           
  Misses        636      636           
  Partials       24       24           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@r-hang r-hang merged commit bc9fb4d into main May 14, 2024
7 of 8 checks passed
@r-hang r-hang deleted the rhang/fix-golangci branch May 14, 2024 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants