-
Notifications
You must be signed in to change notification settings - Fork 48
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
Normalize META.yml indentation #2209
Comments
Yea I think there was issue where PRs create huge diff after the indentation changes. #2207 all the gecko updates have less indentation than the default one. Let me find a way to enforce the format in the lint |
Oh, does the Gecko sync bot not use the API for updating metadata then? |
nope I think they use a separate script. I am unsure what YAML library they use, which leads to a difference in formatting |
It's PyYAML: (cc @jgraham) I also don't quite understand the indentation rules that the Go library is using, either… It seems to create an indent of four at the first level, and then two afterwards? |
When working on #2196 I noticed different indentation in different files, which made it harder to work with the files.
Could we normalize the formatting of META.yml files and enforce that formatting in the lint?
@KyleJu WDYT?
P.S. I thought there was already an issue about this but I couldn't find it. Sorry if it is nevertheless a dupe of some existing issue.
The text was updated successfully, but these errors were encountered: