Skip to content
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

fix: sass dependency list referencing source file in win32 #621

Merged
merged 1 commit into from
Dec 18, 2023

Conversation

kaisermann
Copy link
Member

@kaisermann kaisermann commented Dec 17, 2023

Should close #619

Argh, just remembered that this is hard to reproduce in tests for some reason: https://github.com/sveltejs/svelte-preprocess/blob/main/src/transformers/scss.ts#L72

@kaisermann kaisermann changed the title test: add missing assertions for scss deps fix: sass dependency list referencing source file in win32 Dec 17, 2023
@kaisermann kaisermann marked this pull request as ready for review December 17, 2023 15:42
@kaisermann kaisermann self-assigned this Dec 17, 2023
@kaisermann kaisermann merged commit 209312f into main Dec 18, 2023
20 checks passed
@kaisermann kaisermann deleted the sass-ix-619 branch December 18, 2023 09:06
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.

Warnings when using SASS: "svelte.preprocess returned this file as a dependency of itself"
1 participant