-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: Allow custom syntax #47
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.
There's a merge conflict.
There are currently no updates in tests for |
I didn't include any additional tests for I did add a test that shows the new option throws when invalid. |
Co-authored-by: Milos Djermanovic <milos.djermanovic@gmail.com>
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, thanks!
Prerequisites checklist
What is the purpose of this pull request?
Adds support for custom syntax to the parser.
What changes did you make? (Give an overview)
languageOptions.customSyntax
toCSSLanguage
CSSLanguage
and rules/syntax
entrypoint for common custom syntax like TailwindRelated Issues
fixes #37
Is there anything you'd like reviewers to focus on?
There is a limitation in CSSTree that prevents custom functions like Tailwind's
theme()
to work properly, so I left that out:csstree/csstree#292
When that issue is resolved, we can revisit.