Skip to content
This repository has been archived by the owner on Nov 21, 2023. It is now read-only.

JS file format error #33

Closed
SOVLOOKUP opened this issue Jun 28, 2021 · 3 comments
Closed

JS file format error #33

SOVLOOKUP opened this issue Jun 28, 2021 · 3 comments

Comments

@SOVLOOKUP
Copy link

SOVLOOKUP commented Jun 28, 2021

Some tools like rollup will run incorrectly(err) when loading js files because of this line:
https://github.com/leafac/caxa/blob/master/src/index.ts#L1

Can this line be removed?

@leafac
Copy link
Owner

leafac commented Jun 29, 2021

That’s the line that allows you to run caxa as a command-line tool. It can’t be removed, but I guess it could be moved into its own file. But caxa is probably far from being the only tool in this situation. Did you try to look into Rollup & Skypack to see what’s their take on this?

Also, I’m curious: Why do you need to bundle caxa? It’s usually a development dependency and it only works in Node.js, not the browser. Are you trying to do some sort of packaging-as-a-service?

@leafac
Copy link
Owner

leafac commented Jul 11, 2021

How did you end up solving this?

@SOVLOOKUP
Copy link
Author

I just modified the file in node_module, thank you for your help~

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants