-
Notifications
You must be signed in to change notification settings - Fork 337
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
Avoid using default exports #175
Conversation
rollup.config.ts
Outdated
@@ -17,7 +17,7 @@ const plugins = [ | |||
]; | |||
|
|||
export default defineConfig([{ | |||
input: "lib/index.standalone.ts", | |||
input: "lib/index.ts", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since the input
is now the same for all files, we can put it in a variable so it can be re-used.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Actually have some things that can go into this PR, let me create some changes. |
See #176 |
I am going to take a look and see if we can somehow keep the UMD bundle compatible. I agree that we should avoid breaking this if possible. |
@frederikprijck created #181 to handle this, I think for the sake of backwards compatibility keeping a separate index for UMD is alright. The named exports API also seems really quirky for this one. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@frederikprijck @adamjmcgrath shall we merge this in so work can continue on #174? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, pushed a commit to fix the file reference in static/index.html
Co-authored-by: Jon Koops <jonkoops@gmail.com> Co-authored-by: Ewan Harris <ewan.harris@okta.com>
Description
Drops the default exports, both from the internal and the external API.
This should simplify support for CJS and ESM, see #174 .
Checklist