-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added a github workflow, that runs yamllint on our deployment scripts #615
Conversation
Can we rename the yamllint config file so that it is not hidden. |
I'd like to keep the extra spaces after colons. Disabling the
|
If this works, should we promote it to the top level of the whole project ? Possibly leave as-is for this PR, test it out and then apply it to everything in a separate PR. |
We probbably should at some point, though when I ran it the first time it produced several errors, so there would be lots of cleaning up to do. |
Ok this has been changed in commit: df8326b |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a good idea. We should look to do more of this.
No description provided.