-
Notifications
You must be signed in to change notification settings - Fork 90
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
Add @use member usage and load error tests #1349
Conversation
This also cleans up the existing specs and adds a few for missing cases.
Consolidate and flesh out @import load specs
You only need to review the commits after the merge; everything else was already reviewed when it landed on |
@function member() {@return from other2} | ||
|
||
<===> conflict/error | ||
Error: Multiple global modules have a function named "member". |
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.
I don't see where this error is coming from in dart-sass but could it include the names of the modules that have conflicting functions? It would make it easier to track down when many modules are used.
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 already a TODO for that: https://github.com/sass/dart-sass/blob/5a64dec4ec54ef4fc8fdf0ca0c16a83fec81463b/lib/src/async_environment.dart#L589
================================================================================ | ||
<===> conflict/extension/scss_and_css/input.scss | ||
// This import can't be resolved because it could refer to either the ".sass" or | ||
// ".css" file. |
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.
Same question here as I added to sass/dart-sass#603. Might cause confusion on your second build?
[skip dart-sass]