Add strictOrder option#88

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Draft

matyunya wants to merge2 commits intosveltejs:master

base:master

Choose a base branch

frommatyunya:master

Draft

Add strictOrder option#88

matyunya wants to merge2 commits intosveltejs:masterfrommatyunya:master

Conversation

Copy link

matyunya commentedJan 7, 2020•
edited by lukeed
Loading

Following this discussion:#86.

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

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

matyunyaand others added2 commits

January 7, 2020 19:31

add strictOrder option

89ef70f

Merge remote-tracking branch 'upstream/master'

45e6e4c

Copy link

Member

antony commentedOct 14, 2020

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

antony added this to the7.0.0 milestone

Oct 14, 2020

Copy link

Member

lukeed commentedOct 19, 2020

As@antony said, this requires coordination withsvelte directly.

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

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