Skip to content

chore(core): changes ldml tsconfig to es2020 🙀#7328

Merged
mcdurdin merged 2 commits intofeature-ldmlfrom
chore/core/epic-ldml/tsconfig-to-esm
Sep 20, 2022
Merged

chore(core): changes ldml tsconfig to es2020 🙀#7328
mcdurdin merged 2 commits intofeature-ldmlfrom
chore/core/epic-ldml/tsconfig-to-esm

Conversation

@mcdurdin
Copy link
Copy Markdown
Member

Also splits the root tsconfig.json into tsconfig.cjs.json and tsconfig.esm.json. These are not typically used at present but in theory you could do a tsc -b <tsconfig.x.json> and build all modules. In practice, we still have some prebuild steps that make that a bit of a pain.

I have noted some inconsistency with tsc -b when referenced modules have different module modes, so splitting them seems wise. Also helpful for a simple index of which modules are in which format...

@keymanapp-test-bot skip

Also splits the root tsconfig.json into tsconfig.cjs.json and
tsconfig.esm.json. These are not typically used at present but in theory
you could do a `tsc -b <tsconfig.x.json>` and build all modules. In
practice, we still have some prebuild steps that make that a bit of a
pain.
@keymanapp-test-bot
Copy link
Copy Markdown

keymanapp-test-bot bot commented Sep 19, 2022

Base automatically changed from chore/developer/epic-ldml/7301-no-checksums-please-kmc to feature-ldml September 19, 2022 23:31
@mcdurdin
Copy link
Copy Markdown
Member Author

note: jenkins build failure addressed in #7311.

@mcdurdin mcdurdin merged commit e5c7526 into feature-ldml Sep 20, 2022
@mcdurdin mcdurdin deleted the chore/core/epic-ldml/tsconfig-to-esm branch September 20, 2022 03:42
Copy link
Copy Markdown
Member

@srl295 srl295 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pmlgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants