-
Notifications
You must be signed in to change notification settings - Fork 76
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
Sass compiler does not throw an error when expected #10188
Comments
This is a known limitation for unknown Sass functions:
Stylelint's Keeping track of custom functions will be a manual process for now, but we may be able to automate this. |
**Related Issue:** #10188 ## Summary The Sass compiler does not error on unknown functions, but we can use Stylelint's [`function-no-unknown`](https://github.com/stylelint-scss/stylelint-scss/tree/master/src/rules/function-no-unknown) to ensure function names are properly referenced.
…int (#10313) **Related Issue:** #10188 ## Summary This adds a script that tracks all custom Sass functions and updates the metadata for Stylelint’s [`function-no-unknown`](https://github.com/stylelint-scss/stylelint-scss/tree/master/src/rules/function-no-unknown) rule, addressing workflow improvements as discussed in #10188 (comment). Our pre-commit hook will run this script if there is a staged Sass file and will add any changes (if any). **Note**: this depends on #10311. --------- Co-authored-by: Ben Elan <no-reply@benelan.dev>
This step has ben automated via a |
Installed and assigned for verification. |
Priority impact
p - low
Summary
Sass names refactor in the
loader
revealed that Sass compiler did not throw an error when it was referencing a function that does not exist.Desired Outcome
Sass compiler should throw an error when referencing a function that does not exist.
Resources
No response
The text was updated successfully, but these errors were encountered: