Skip to content

Don't use @import in the legacy importer#342

Merged
nex3 merged 2 commits intomainfrom
no-legacy-import
Oct 29, 2024
Merged

Don't use @import in the legacy importer#342
nex3 merged 2 commits intomainfrom
no-legacy-import

Conversation

@nex3
Copy link
Contributor

@nex3 nex3 commented Oct 22, 2024

This unfortunately may make error reports look a little odd for errors
on the first line of SCSS files loaded via the legacy importer and
it'll give indented-syntax errors an off-by-one error, but there's an
easy fix: stop using the legacy API.

Closes #340

@nex3 nex3 requested a review from jathak October 22, 2024 20:02
This unfortunately may make error reports look a little odd for errors
on the first line of SCSS files loaded via the legacy importer and
it'll give indented-syntax errors an off-by-one error, but there's an
easy fix: stop using the legacy API.

Closes #340
@nex3 nex3 force-pushed the no-legacy-import branch from fe7cb48 to 0ec616e Compare October 22, 2024 20:02
@nex3 nex3 changed the title Export the SassBoolean type Don't use @import in the legacy importer Oct 22, 2024
Co-authored-by: Jennifer Thakar <jathak@google.com>
nex3 added a commit to sass/dart-sass that referenced this pull request Oct 24, 2024
nex3 added a commit to sass/dart-sass that referenced this pull request Oct 24, 2024
@nex3 nex3 merged commit 80e2b88 into main Oct 29, 2024
@nex3 nex3 deleted the no-legacy-import branch October 29, 2024 01:04
nex3 added a commit to sass/dart-sass that referenced this pull request Oct 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Using legacy APIs with importers triggers @import deprecation warning

2 participants