-
Notifications
You must be signed in to change notification settings - Fork 176
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
Use consistent stylesheet indentation #423
Conversation
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.
I don't think this is worth doing unless we add Prettier or some other automatic formatting. Otherwise, things will get messy right away again.
Also, which config to use is quite opinionated. Maybe the default Prettier config is the one to go with, as it should be the least opinionated.
Update: this came up during the workgroup meeting. The agreement is to add Prettier as part of this PR and tackle automating the checks in a separate task. Also merging should wait for #344 to be merged first. |
eed3aad
to
dbfe740
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.
LGTM once the merge conflicts are fixed
28265c7
to
818c412
Compare
@swift-ci please test |
@alexandersandberg I went with the absolute minimum Prettier config and re-run it on all files again. |
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.
👍
@swift-ci please test |
Motivation:
Stylesheet source files used a mixture of 2-space and 4-space based indentation.
Modifications:
Make all stylesheet files use 2-space indentation.