Skip to content

fix: use isolatedModules: true in tsconfig.json #469

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Feb 10, 2022

Conversation

lance
Copy link
Member

@lance lance commented Feb 9, 2022

This setting ensures the module can be used in projects where the workflow
includes type checking and transpilation as two separate steps.

See: https://ncjamieson.com/dont-export-const-enums/

Fixes: #456

Signed-off-by: Lance Ball lball@redhat.com

This setting ensures the module can be used in projects where the workflow
includes type checking and transpilation as two separate steps.

See: https://ncjamieson.com/dont-export-const-enums/

Fixes: cloudevents#456

Signed-off-by: Lance Ball <lball@redhat.com>
@lance lance requested a review from a team February 9, 2022 20:17
@lance lance added the type/fix A change that fixes something that is broken label Feb 9, 2022
Copy link
Member

@grant grant left a comment

Choose a reason for hiding this comment

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

Formatting changes should be separate. Otherwise lgtm

Signed-off-by: Lance Ball <lball@redhat.com>
@lance lance merged commit b5c0b56 into cloudevents:main Feb 10, 2022
@lance lance deleted the lance/isolated-modules branch February 10, 2022 13:39
@jketcham
Copy link

Thanks for jumping on this fix! Any idea when the next release will be?

@lance
Copy link
Member Author

lance commented Feb 10, 2022

@jketcham we're pretty loose about the timing of things. I'd like to land #471 before another release, but assuming that's relatively non-controversial, I'd imagine we could have a release out next week at the latest.

@jketcham
Copy link

@lance sounds good 👍

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
type/fix A change that fixes something that is broken
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can not build with --isolatedModules
3 participants