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

chore: Build multiple modules instead of one index.js #7

Merged
merged 3 commits into from
Feb 11, 2021

Conversation

wilsonshakespeare
Copy link
Owner

  • refactor: Move build command to build.sh
    note: this will allow multiple module build entry files

  • feat: Add indexify.js script, this will generate moduleName.js files
    note: this is because tsdx.config.js override do not generate js files

  • feat: Add tsdx.config.js to generate multiple esm, cjs modules

  • refactor: Move index file out from module folders
    refer Multiple entries to output multiple build files jaredpalmer/tsdx#175 (comment)

Important note:
Base on the comment tsdx do not support multiple entry

- refactor: Move build command to build.sh
  note: this will allow multiple module build entry files

- feat: Add indexify.js script, this will generate moduleName.js files
  note: this is because tsdx.config.js override do not generate js files

- feat: Add tsdx.config.js to generate multiple esm, cjs modules

- refactor: Move index file out from module folders
  refer jaredpalmer/tsdx#175 (comment)

Important note:
Base on the comment tsdx do not support multiple entry
@github-actions
Copy link

github-actions bot commented Feb 7, 2021

size-limit report 📦

Path Size
dist/utilzed.cjs.production.min.js 0 B (-100% 🔽)
dist/utilzed.esm.js 0 B (-100% 🔽)
dist/index.cjs.production.min.js 3.15 KB (+100% 🔺)
dist/index.esm.js 3.17 KB (+100% 🔺)
dist/promise.cjs.production.min.js 2.5 KB (+100% 🔺)
dist/promise.esm.js 2.5 KB (+100% 🔺)
dist/timeconvert.cjs.production.min.js 287 B (+100% 🔺)
dist/timeconvert.esm.js 292 B (+100% 🔺)
dist/validation.cjs.production.min.js 337 B (+100% 🔺)
dist/validation.esm.js 342 B (+100% 🔺)

@wilsonshakespeare wilsonshakespeare merged commit 32b7ed0 into master Feb 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant