Skip to content

docs: mention module: "ES2020" compatibility#376

Merged
ezolenko merged 1 commit into
ezolenko:masterfrom
agilgur5:docs-module-es2020
Jul 6, 2022
Merged

docs: mention module: "ES2020" compatibility#376
ezolenko merged 1 commit into
ezolenko:masterfrom
agilgur5:docs-module-es2020

Conversation

@agilgur5

@agilgur5 agilgur5 commented Jul 4, 2022

Copy link
Copy Markdown
Collaborator

Summary

Update docs and check-tsconfig error message to mention module: "ES2020" compatibility

Details

@agilgur5 agilgur5 added scope: docs Documentation could be improved. Or changes that only affect docs kind: internal Changes only affect the internals, and _not_ the public API or external-facing docs labels Jul 4, 2022
- it's already been supported since eb1dd17, but the docs and error message were not updated to mention it
  - so add both to make sure users aren't confused
  - also re-order it to be ES2015, ES2020, then ESNext consistently, which is their module order (c.f. microsoft/TypeScript#24082)

- modify test to account for the new error message
  - make it a bit more resilient to change by only testing a substring as well (c.f. https://jestjs.io/docs/expect#tothrowerror)
@agilgur5 agilgur5 force-pushed the docs-module-es2020 branch from a4bfc28 to 69c196c Compare July 4, 2022 14:46
@ezolenko ezolenko merged commit 9c50898 into ezolenko:master Jul 6, 2022
@agilgur5 agilgur5 added kind: dx Improvements to dev experience, e.g. error messages, logging, external-facing docs, etc and removed kind: internal Changes only affect the internals, and _not_ the public API or external-facing docs labels Jul 22, 2022
@agilgur5 agilgur5 deleted the docs-module-es2020 branch July 2, 2023 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind: dx Improvements to dev experience, e.g. error messages, logging, external-facing docs, etc scope: docs Documentation could be improved. Or changes that only affect docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants