woodpecker-ci / woodpecker

Woodpecker is a simple, yet powerful CI/CD engine with great extensibility.
https://woodpecker-ci.org
Apache License 2.0
4.3k stars 371 forks source link

Add `cli lint` option to treat warnings as errors #4373

Closed 6543 closed 1 week ago

6543 commented 1 week ago

Add option treat warnings as errors for pipeline linter:

https://ci.woodpecker-ci.org/repos/3780/pipeline/22592/34 FAILED SUCCESSFULLY

6543 commented 1 week ago

should fail and pass after #4372 merged ...

woodpecker-bot commented 1 week ago

Deployment of preview was successful: https://woodpecker-ci-woodpecker-pr-4373.surge.sh

6543 commented 1 week ago

https://ci.woodpecker-ci.org/repos/3780/pipeline/22592/34 FAILED SUCCESSFULLY

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 27.36%. Comparing base (4ed5e4e) to head (f1f3d35). Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
cli/lint/utils.go 0.00% 2 Missing :warning:
cli/exec/exec.go 0.00% 1 Missing :warning:
cli/lint/lint.go 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #4373 +/- ## ========================================== - Coverage 27.37% 27.36% -0.02% ========================================== Files 380 380 Lines 27781 27781 ========================================== - Hits 7606 7601 -5 - Misses 19490 19495 +5 Partials 685 685 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.