Skip to content

Add support for TypeScript 5.0 #264

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

Merged
merged 8 commits into from
Apr 6, 2023
Merged

Conversation

bicknellr
Copy link
Contributor

@bicknellr bicknellr commented Apr 5, 2023

  • Adds typescript@~5.0 to the list of TS versions that this package is tested on.
  • Adds support for the new AST location of decorators as of typescript@4.8.
  • Handles the removal of isTypeAssertion (compiler API) and DocumentAndElementEventListeners (TS's DOM types).
  • Handles changes to JSDoc description text indentation.

bicknellr and others added 7 commits April 5, 2023 15:54
Somewhere between 3.8 and 5.0, indentation for JSDoc description lines starting
with whitespace was fixed to not include the additional space immediately after
the line's leading `*`. The indentiation in this test causes it to fail in at
least TS 5.0, so I'm removing it rather than trying to track down exactly when
this change was made and then propagating the version number to the test.
@bicknellr bicknellr force-pushed the typescript-compat branch from e14219b to e85927f Compare April 5, 2023 22:55
@bicknellr bicknellr marked this pull request as ready for review April 5, 2023 22:59
@rictic rictic merged commit fa548cd into runem:1.1 Apr 6, 2023
@bicknellr bicknellr deleted the typescript-compat branch April 7, 2023 20:21
@bicknellr
Copy link
Contributor Author

For reference, this is the same error I was seeing before I downgraded package-lock.json:
DefinitelyTyped/DefinitelyTyped#63431

@bicknellr
Copy link
Contributor Author

That issue might be for lit-analyzer actually.

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.

2 participants