Use "lib" for published JavaScript code
Opened this issue · 2 comments
ericcornelissen commented
Initiative
Description
Currently, most of the packages compile into an "outDir"
named "build". This is fine in theory - as it is entirely abstracted from the package user - but a clearer and more standard name would still be preferable.
Notes
- I will update this for each package when releases happens.
- For some package, just changing the
tsconfig.json
file is enough but for some more changes are required. For example to keep the internal dependencies working or to keep the package exports working.
Progress
- benchmarking
- cli (ff82501)
- core (ff82501)
- language-css (3fbd58f)
- language-html (1c927f7)
- language-js (a87c7f4)
- language-utils (ec3c06b)
- mangler-css-classes (#368)
- mangler-css-variables (#368)
- mangler-html-attributes (#368)
- mangler-html-ids
- mangler-utils (2cb37e2)
- testing (fdae356)
- types (af58010)
Followup
- Remove caching of the
packages/*/build
folder in CI (example) - Unignore
**/build/
in the TypeScript build configuration - Unignore
build/
in the ESLint configuration - Remove
build
as a compiled directory from.values.cjs
Related
As this issue concerns all package, the labels of individual packages won't be added.
ericcornelissen commented
The nyc configuration file needs to be updated to exclude the lib/
directory when creating a coverage report.
EDIT: This is fixed in a87c7f4
ericcornelissen commented