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

Add strictOrder option #88

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Add strictOrder option #88

wants to merge 2 commits into from

Conversation

matyunya
Copy link

@matyunya matyunya commented Jan 7, 2020

Following this discussion: #86.

New option to specify preprocessors order. Issue: sveltejs/svelte#4141.

PR for Svelte itself: sveltejs/svelte#4222 sveltejs/svelte#4282.

@antony
Copy link
Member

antony commented Oct 14, 2020

This depends on sveltejs/svelte#4282 so can't currently be merged

@antony antony added this to the 7.0.0 milestone Oct 14, 2020
@lukeed
Copy link
Member

lukeed commented Oct 19, 2020

As @antony said, this requires coordination with svelte directly.

If older versions of the compiler won't throw an error on the unknown strictOrder option, then this can land at any point during the 7.x timeline. Otherwise, we'll have to raise the required svelte peer version, which will will be "breaking" & require a new major version.

Removing this from the 7.0 milestone for that reason, but I think we should definitely work towards this.

@lukeed lukeed removed this from the 7.0.0 milestone Oct 19, 2020
@benmccann benmccann marked this pull request as draft January 19, 2021 21:24
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

4 participants