Skip to content

Fix CSScomb when Sass tmLanguage installed#52

Merged
tonyganch merged 1 commit intocsscomb:masterfrom
karlhorky:patch-1
Feb 3, 2015
Merged

Fix CSScomb when Sass tmLanguage installed#52
tonyganch merged 1 commit intocsscomb:masterfrom
karlhorky:patch-1

Conversation

@karlhorky
Copy link
Contributor

CSScomb was not running when Sass tmLanguage was used for syntax highlighting.

Fixes #18

CSScomb.py Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, wow, thank you!
But you know what, there should actually be def is_sass() returning sass as a syntax name, not scss.

The Sass tmLanguage package can be used to highlight SCSS and Sass files. If
used on an SCSS file, CSSComb does not run (since it does not match the scope
name as starting with `source.scss`).

This commit enables CSSComb for both Sass and SCSS files when being highlighted
with the Sass tmLanguage package.

Fixes #18
@karlhorky
Copy link
Contributor Author

@tonyganch No problem, glad to help :)

Please review again. I've added is_sass() as well to support Sass files as well as SCSS files using the Sass tmLanguage package.

tonyganch added a commit that referenced this pull request Feb 3, 2015
Fix CSScomb when Sass tmLanguage installed
@tonyganch tonyganch merged commit 9b6467d into csscomb:master Feb 3, 2015
@tonyganch
Copy link
Member

Thank you! 🐘

@karlhorky karlhorky deleted the patch-1 branch February 3, 2015 23:15
@karlhorky
Copy link
Contributor Author

Thanks for the merge :bowtie:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Doesn't run in .scss files

2 participants