Skip to content

drop-in replacement for importFrom/exportTo #734

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

Conversation

romainmenke
Copy link
Member

@romainmenke romainmenke commented Nov 29, 2022

What I did :

  • copy/paste the existing plugins as they are now on the main branch
  • removed all the "polyfill/fallback" behaviour
  • added importedStylesOverrideDocumentStyles as plugin option to all plugins
  • removed overrideImportFromWithRoot from custom-properties

I picked a bad name in overrideImportFromWithRoot.
It is very confusing what is overriding what purely from the name.
I hope importedStylesOverrideDocumentStyles is clearer.

@Antonio-Laguna
Copy link
Member

After thorough consideration, we don't think this is the right way to move forward. We still strongly believe that there are better ways to solve the issues where importFrom and exportTo are used today. We will keep our decision to get rid of importFrom and exportTo from the postcss-preset-env ecosystem starting with version 8 and upwards.

Our main concern is that we have limited resources to create and maintain all the plugins here.

We have separated the removed functionality into separate plugins and the source code for this is available to anyone who wants to take over ownership.

We'll keep this visible for some reasonable time and then we will just close it.

@romainmenke
Copy link
Member Author

Going to close this.
We can always restore if needed later.

@romainmenke romainmenke deleted the drop-in-replacement-for-import-export--courageous-black-widow-spider-fc53e36515 branch January 20, 2023 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants