chart-testing-action icon indicating copy to clipboard operation
chart-testing-action copied to clipboard

yamllint not reporting line numbers

Open IanMoroney opened this issue 3 years ago • 3 comments

I noticed that ct lint calls yamllint to validate Chart.yaml, but when it spots a warning or an error, it doesn't tell you which line it is:

>>> yamllint --config-file /home/runner/_work/_tool/ct/v3.3.0/x86_64/etc/lintconf.yaml mychart/Chart.yaml
Error: [comments] too few spaces before comment
Error: Error linting charts: Error processing charts

Conversely, yamllint itself in its default configuration reports the line number:

yamllint b2c-perf-test-client/Chart.yaml 
b2c-perf-test-client/Chart.yaml
  1:1       warning  missing document start "---"  (document-start)
  26:14     warning  too few spaces before comment  (comments)

Either: How do we get ct lint to report line numbers, or Can you configure lintconf.yaml to report line numbers by default ?

Additionally, it may be the case where your lintconf.yaml is reporting warnings as errors, as yamllint 1.26.1 only reported the comment issue as a warning and not an error.

IanMoroney avatar Jun 21 '21 18:06 IanMoroney

@IanMoroney This may get more responses by creating over in the main ct repo: https://github.com/helm/chart-testing

tylerauerbeck avatar Jul 25 '21 07:07 tylerauerbeck

This issues seems a clone of https://github.com/helm/chart-testing-action/issues/67

gcaracuel avatar Aug 02 '21 08:08 gcaracuel

The line numbers missing is an artefact of the auto output format picking up it is running in a Github Action and that output format not having the file names or line numbers on there. A design decision of yamllint itself.

https://github.com/adrienverge/yamllint/blob/85ccd625a382423edd988fcb796c4dd5897886fb/yamllint/cli.py#L112-L114

Forcing the output format of yamllint to standard would get you the output you seek.

Rotonen avatar Sep 09 '21 13:09 Rotonen

closing as inactive

cpanato avatar Mar 24 '23 11:03 cpanato