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

Drop webpack build in favor of tsc #29

Closed
AuHau opened this issue Mar 8, 2021 · 1 comment
Closed

Drop webpack build in favor of tsc #29

AuHau opened this issue Mar 8, 2021 · 1 comment
Labels
issue wontfix This will not be worked on

Comments

@AuHau
Copy link
Contributor

AuHau commented Mar 8, 2021

Webpack is not necessary for building nodejs only sources. Typescript itself ships with tsc compiler that could be used.

@bee-runner bee-runner bot added the issue label Mar 8, 2021
@nugaon
Copy link
Member

nugaon commented Mar 9, 2021

It seems to me rather a drawback, than a feature that we need.
Webpack is a common tool among Swarm JS projects, which fits completely with our needs.
It provides many customizable properties that we can utilize to examine/alter/mutate/enchant our code.
Webpack uses babel now like Jest too.
If we build browser code with webpack and we don't do with that for node, then it is an asymmetric workflow.
I think we had better to keep the current webpack environment for compiling both node and browser boundles.
Regardless of these, I support to review the webpack configuration since it surely has unnecessary settings.

@Cafe137 Cafe137 added the wontfix This will not be worked on label May 10, 2021
@Cafe137 Cafe137 closed this as completed May 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants