Skip to content

add --warnings-as-errors flag to flow cadence lint#2290

Open
holyfuchs wants to merge 1 commit intomasterfrom
holyfuchs/lint-warnings-as-errors
Open

add --warnings-as-errors flag to flow cadence lint#2290
holyfuchs wants to merge 1 commit intomasterfrom
holyfuchs/lint-warnings-as-errors

Conversation

@holyfuchs
Copy link
Member

Enables CI pipelines to fail on lint warnings without shell hacks like grepping output or checking for warning strings.
Pass --warnings-as-errors and the exit code will be non-zero if any warnings are found.

@github-actions
Copy link

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/cadence/lint.go 75.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

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.

2 participants