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
Introduce style guide #88
Conversation
de8a01a
to
82e4c0d
Compare
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.
As this also includes documentation about the Asciidoc style should there be some words about the usage of vale and which rules are enforced by it?
Other than that, I found some small things to fix, but overall LGTM.
Good point. Will do. |
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.
The typo needs to be fixed. Other than that: LGTM
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.
Just a few minor things. The rest looks good and sensible.
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.
A bunch of typos, but LGTM overall now
Co-authored-by: Simon Rüegg <simon.ruegg@vshn.ch>
Co-authored-by: Tobias Brunner <tobias.brunner@vshn.ch>
Co-authored-by: Simon Gerber <simon.gerber@vshn.ch>
Use function and template function instead of method and classes.
Co-authored-by: Simon Rüegg <simon.ruegg@vshn.ch>
Co-authored-by: Simon Gerber <simon.gerber@vshn.ch>
Co-authored-by: Simon Gerber <simon.gerber@vshn.ch>
We decided in projectsyn/documentation#88 that we want to pad arrays in Jsonnet. This commit adds the `--pad-arrays` argument to the default jsonnetfmt arguments in the component template Makefile.
We decided in projectsyn/documentation#88 that we want to pad arrays in Jsonnet. This commit adds the `--pad-arrays` argument to the default jsonnetfmt arguments in the component template Makefile.
This documents the style guide as it was discussed in #40.
Checklist