You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With @import, it's possible for a file to refer to a mixin or function that gets redefined later on. Right now References will just override the previous definition and treat it as though it always referred to whatever the last value was, but the more correct behavior would be to throw an error, since changing the definition isn't supported in the module system.
This error was already thrown when different results were generated
due to different entrypoints, but not for the same entrypoint. In
practice I don't think this can come up with the module migrator,
though, due to #99.
Given that we're getting ready to start properly deprecating @import and this doesn't seem to be causing much user pain, I'm going to close it out as not planned.
With
@import
, it's possible for a file to refer to a mixin or function that gets redefined later on. Right nowReferences
will just override the previous definition and treat it as though it always referred to whatever the last value was, but the more correct behavior would be to throw an error, since changing the definition isn't supported in the module system.For example:
I don't think this is worth fixing before we launch, though.
The text was updated successfully, but these errors were encountered: