-
Notifications
You must be signed in to change notification settings - Fork 357
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
Merge and enable feature.use 🎉 #835
Conversation
Merge master into feature.use
Merge master into feature.use
Add load-css()
Merge origin/master into feature.use
Merge master into feature.use
Merge master into feature.use
Merge master into feature.use
We had been using null to represent an empty configuration, but that caused problems when an empty map snuck its way in as well. Now we always use maps, with a const empty map for the common case. See sass/sass#2744
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.
Glad to see this day come, congrats!
I left two comments.
migrate your stylesheets to the new module system! | ||
|
||
[migrator blog]: https://sass-lang.com/blog/7858341-the-module-system-is-launched | ||
[Sass migrator]: https://sass-lang.com/documentation/cli/migrator |
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.
Just want to warn you that the link for the migrator isn't live yet.
dart_test.yaml
Outdated
# Indicates tests that use the new module system, and so should only run on | ||
# the `feature.use` branch. | ||
module-system: | ||
|
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.
Should these be removed?
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.
Good call, deleted.
Closes #554
See sass/sass-spec#1468