Skip to content
This repository was archived by the owner on Aug 12, 2024. It is now read-only.

Fixes make lint when run without args#636

Merged
m1kola merged 1 commit into
operator-framework:mainfrom
m1kola:make_lint_fix
Jun 14, 2023
Merged

Fixes make lint when run without args#636
m1kola merged 1 commit into
operator-framework:mainfrom
m1kola:make_lint_fix

Conversation

@m1kola
Copy link
Copy Markdown
Member

@m1kola m1kola commented Jun 14, 2023

In #631 I added an optional GOLANGCI_LINT_ARGS into make lint, but... When you run make lint you get this:

ERRO Running error: context loading failed: failed to load packages: package .: no go files to analyze

This should fix it.

Example runs:

make lint
make lint GOLANGCI_LINT_ARGS="--verbose"
make lint GOLANGCI_LINT_ARGS="--out-format github-actions"

Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com>
@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 14, 2023
@m1kola m1kola changed the title Fixes make lint when run without args Fixes make lint when run without args Jun 14, 2023
@m1kola m1kola marked this pull request as ready for review June 14, 2023 15:57
@m1kola m1kola requested a review from a team as a code owner June 14, 2023 15:57
@openshift-ci openshift-ci Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 14, 2023
@m1kola m1kola merged commit 4a597b7 into operator-framework:main Jun 14, 2023
@m1kola m1kola deleted the make_lint_fix branch June 14, 2023 17:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants