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.

Sign up for GitHub

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

Jump to bottom

Remove cjs output in @exile-watch/rollup-config #1

Open
sbsrnt opened this issue Feb 25, 2024 · 0 comments
Open

Remove cjs output in @exile-watch/rollup-config #1

sbsrnt opened this issue Feb 25, 2024 · 0 comments
Labels
splinters/rollup-config @exile-watch/rollup-config

Comments

@sbsrnt
Copy link
Member

sbsrnt commented Feb 25, 2024 •
edited
Loading

Caution

This can only be done once all packages that utilize rollup-config and crucible itself are properly tested (unit tests, e2e tests, maybe regression tests?)

User story
I would like receive a lower bundle size to save my bandwidth.

Whenever I visit exile.watch I have to download content.
Part of this content is coming from packages that are bundled with https://github.com/exile-watch/splinters/pkgs/npm/rollup-config.

Currently the output contains .esm and .cjs format.
exile.watch does not support unmaintained browser versions or browsers that do not support es modules .

Acceptance criteria

  • Remove cjs output that is defined here:
  • Remove sourcemap: true from esm output
@sbsrnt sbsrnt added the splinters/rollup-config @exile-watch/rollup-config label Feb 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
splinters/rollup-config @exile-watch/rollup-config
Projects
splinters (build tools)
Status: No status
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sbsrnt