drone / proposal

Drone Project Design Documents
13 stars 4 forks source link

Make the linter error messages more helpful #29

Open WillGibson opened 2 years ago

WillGibson commented 2 years ago

When the linter fails on an error in the YAML configuration it just says things like...

linter: duplicate step names

When the linter detects an error, it presumably knows the context, so could include those details, e.g...

linter: duplicate step names ("step-name-goes-here", "another-one-goes-here-if-more-than-one")
WillGibson commented 2 years ago

Another classic example :-)

linter: invalid or unknown pipeline dependency
WillGibson commented 2 years ago

Another example...

yaml: line 51: did not find expected key

I think that's line 51 of one of the pipelines, but I don't know which pipeline easily or what line it is in the file.

MarieReRe commented 2 years ago

👋🏻 Hey @WillGibson thanks for the proposal. I just wanted to update you, the team is currently backed up on these and hopefully get back to these soon. Feel free to join our community on slack as well if you want to stay up to date!