-
Notifications
You must be signed in to change notification settings - Fork 803
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
YAML indentation guideline #622
Comments
It seems like the most common practice is using the parent1 kind of indentation. Going with that! |
Do you know what kubernetes/charts repo does? They might have a YAML
template linter we might want to adopt.
…On Sun, Apr 1, 2018 at 8:57 AM, Erik Sundell ***@***.***> wrote:
In the yaml templates, we are currently mixing a lot in between the
following ways. I'd like to stop thinking about what I'd do myself - can we
decide on a guideline to aim for?
parent1:
- key1: value1
- key2: value2
parent2:
- key1: value1
- key2: value2
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#622>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAB23qARWcEijlYLLvTqjPaEcR6ENFjJks5tkPjngaJpZM4TC4Ja>
.
--
Yuvi Panda T
http://yuvi.in/blog
|
@yuvipanda I looked at some, prometheus for example, and concluded that the tended to use the first version. I also remember some generated yaml (by chartpress i think) matched that pattern. I'll look for linters and rules setup by kubernetes charts! |
@yuvipanda upon further investigation...
parent2:
- key1: value1
- key2: value2 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In the yaml templates, we are currently mixing a lot in between the following ways. I'd like to stop thinking about what I'd do myself - can we decide on a guideline to aim for?
The text was updated successfully, but these errors were encountered: