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

Add support @TsNoCheck annotation, adds "// @ts-nocheck" to TypeScript #762

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

kohlschuetter
Copy link
Contributor

Typescript would complain about certain errors in the generated file. In some newer versions of TypeScript, an assert that is always true would not be permitted.

We currently have two such asserts in our tests, which prevents us from upgrading the TypeScript version.

Add support for a Java @tsnocheck annotation, which controls the inclusion of the corresponding TypeScript comment. While there is no mandatory package for the annotation, add it to jsweet.lang in core-lib.

Fix tests to use the new annotation.

Typescript would complain about certain errors in the generated file. In
some newer versions of TypeScript, an assert that is always true would
not be permitted.

We currently have two such asserts in our tests, which prevents us from
upgrading the TypeScript version.

Add support for a Java @tsnocheck annotation, which controls the
inclusion of the corresponding TypeScript comment. While there is no
mandatory package for the annotation, add it to jsweet.lang in core-lib.

Fix tests to use the new annotation.
@lgrignon
Copy link
Collaborator

Thanks for contributing @kohlschuetter

@kohlschuetter
Copy link
Contributor Author

Merging broke the build because the core pom version wasn't updated.

See #769, which unbreaks the build.

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