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

Add filenames and line numbers to lint output

Open paddatrapper opened this issue 4 years ago • 2 comments

Is this a request for help?: No


Is this a BUG REPORT or FEATURE REQUEST?: FEATURE REQUEST

The current ct lint output makes it difficult to match the detected issue with where in the templates it is found. Having file names and line numbers for where the issue is detected would make solving them much easier. For example, currently indentation issues are reported as

 Error: [indentation] wrong indentation: expected 3 but found 2

Having an extra at filename.yaml, Line #4 would make it easier to find and fix.

Version of Helm and Kubernetes:

helm - 3.4.0

chart-testing - GitHub Action @ v2.1.0

paddatrapper avatar Nov 29 '21 08:11 paddatrapper

Related: #282

james-d-elliott avatar Dec 03 '21 06:12 james-d-elliott

not adding value, but understanding this error when the change is more than a handful of lines is practically impossible

edit: if you are running in GitHub Actions you can check out the diff of your PR, there will be a message from GitHub Actions on the offending line

dimitarvdimitrov avatar Aug 15 '22 12:08 dimitarvdimitrov

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] avatar Nov 13 '23 01:11 github-actions[bot]

This issue was closed because it has been stalled for 5 days with no activity.

github-actions[bot] avatar Nov 19 '23 01:11 github-actions[bot]