Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Brace style now only lints valid atrule types #635

Merged
merged 3 commits into from
Apr 22, 2016
Merged

Brace style now only lints valid atrule types #635

merged 3 commits into from
Apr 22, 2016

Conversation

bgriffith
Copy link
Member

@bgriffith bgriffith commented Apr 22, 2016 •
edited
Loading

After 1.6 brace-style was linting all atrules to check for single-line statements. Now it only lints functions,if and else.

Closes #634
Closes #632

DCO 1.1 Signed-off-by: Ben Griffith [email protected]

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 96.965% when pulling 81154a0 on bgriffith:feature/1.6-brace-style into e6ca4b5 on sasstools:develop.

@bgriffith bgriffith added this to the 1.6.1 milestone Apr 22, 2016
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 96.982% when pulling c7c27b4 on bgriffith:feature/1.6-brace-style into e6ca4b5 on sasstools:develop.

@DanPurdy
Copy link
Member

Looks good to me 👍

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 96.988% when pulling b62cf0e on bgriffith:feature/1.6-brace-style into ec16676 on sasstools:develop.

@DanPurdy DanPurdy merged commit d98602f into sasstools:develop Apr 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ready for review
Projects
None yet
Milestone
1.6.1
Development

Successfully merging this pull request may close these issues.
3 participants
@bgriffith @coveralls @DanPurdy