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

Feat/sass loader 8 #758

Merged
merged 3 commits into from May 10, 2020
Merged

Conversation

weaverryan
Copy link
Member

@weaverryan weaverryan commented May 9, 2020

Fixes weird test issue for #736 - I'm trying to see if I can debug the "lowest" test failure.

It appears the problem is with Webpack > 4.22 and Vue < 2.5.0.

@weaverryan weaverryan force-pushed the feat/sass-loader-8 branch 6 times, most recently from cdfec57 to 317d728 Compare May 9, 2020 19:20
@weaverryan weaverryan changed the title [Debug] Debugging failures for "Feat/sass loader 8" Feat/sass loader 8 May 9, 2020
@weaverryan
Copy link
Member Author

This is ready!

@Kocal
Copy link
Contributor

Kocal commented May 10, 2020

Nice!

Kocal and others added 3 commits May 10, 2020 13:28
Webpack 4.36.0 is the minimum version that sass-loader 8 requires.

- 4.20.0 has been released in september 2018: https://github.com/webpack/webpack/releases/tag/v4.20.0
- 4.36.0 has been released in july 2019: https://github.com/webpack/webpack/releases/tag/v4.36.0
Fixes an inconsistent test with different Webpack versions
@weaverryan
Copy link
Member Author

Thank you @Kocal!

@weaverryan weaverryan merged commit 2eaa4ad into symfony:master May 10, 2020
@Kocal
Copy link
Contributor

Kocal commented May 10, 2020

Thank you too! :)

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.

None yet

3 participants