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(scoped-elements): remove types.js from exports #2655
base: master
Are you sure you want to change the base?
Conversation
|
Preview published to https://fix-scoped-elements-types-js--open-wc-org.netlify.app/. |
This removes the export map, but are these types still built? That could mean, at least internally, they are still used and that the test results could be a false positive... |
@Westbrook help me understand what's the problem if internal types are still built. You're saying that tests would pass here in CI because those files are available to monorepo packages, but would fail for users that If so, would a repro that installs this version and passes some import tests assuage your concerns? |
Roughly. We should check to see if the files are still being written and if they are, can we stop that and still have this pass? |
https://github.com/bennypowers/scoped-ce-repro-no-typesjs-export using passes with
|
What I did
types.js
from package exports, as we have both types export condition (nodenext) and top-level "types" fallbackReviewer Notes
This may require a major changeset