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

Does not support IIFE format? #788

Closed
Jogiter opened this issue Aug 8, 2020 · 2 comments
Closed

Does not support IIFE format? #788

Jogiter opened this issue Aug 8, 2020 · 2 comments
Labels
kind: feature New feature or request problem: removed issue template OP removed the issue template without good cause solution: unresolved Issue has been closed by OP but root cause has not necessarily been resolved

Comments

@Jogiter
Copy link

Jogiter commented Aug 8, 2020

No description provided.

@Jogiter Jogiter closed this as completed Aug 13, 2020
@Jogiter Jogiter changed the title seems not support rollup format iife seems not support rollup format iife -- use umd Aug 13, 2020
@agilgur5 agilgur5 changed the title seems not support rollup format iife -- use umd Does not support IIFE format? Aug 14, 2020
@agilgur5 agilgur5 added kind: feature New feature or request solution: unresolved Issue has been closed by OP but root cause has not necessarily been resolved problem: removed issue template OP removed the issue template without good cause labels Aug 14, 2020
@agilgur5
Copy link
Collaborator

Please don't remove the issue template, it is there for a reason.

Features required

Yes I don't believe IIFE is supported if used. It wouldn't be that difficult to add though, we'd have to add it to the supported formats here: https://github.com/formium/tsdx/blob/5e5c3f804e68609083be978a080ab905dc7ec19d/src/createBuildConfigs.ts#L46-L89

as well as probably add some special cases into the Rollup config similar to the ones UMD has.

Use-case?

To validate the use case for this feature request I was going to ask if UMD doesn't fit your use-case but based on the title change I guess the answer is yes, UMD can be used instead for your use case.

@StringKe
Copy link

StringKe commented Mar 7, 2022

Is there any temporary solution to support iife?

The generation method of umd is not perfect, and there are currently errors.

#179 (comment)
#759

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: feature New feature or request problem: removed issue template OP removed the issue template without good cause solution: unresolved Issue has been closed by OP but root cause has not necessarily been resolved
Projects
None yet
Development

No branches or pull requests

3 participants