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

fix: support .mts and .cts out of the box #1249

Closed
wants to merge 4 commits into from

Conversation

RebeccaStevens
Copy link

@RebeccaStevens RebeccaStevens commented Aug 28, 2022

Rollup Plugin Name: typescript

This PR contains:

  • bugfix
  • feature
  • refactor
  • documentation
  • other

Are tests included?

  • yes (bugfixes and features will not be merged without tests)
  • no

Breaking Changes?

  • yes (breaking changes will not be merged unless absolutely necessary)
  • no

If yes, then include "BREAKING CHANGES:" in the first commit message body, followed by a description of what is breaking.

List any relevant issue numbers:

Description

Currently out of the box the typescript plugin only supports .ts and .tsx extensions. This PR adds support for .mts and .cts as well without the user needing to modify the includes option.

Additionally only .d.ts were supported as type declaration files, this PR also adds support for .d.mts and .d.cts files.

packages/typescript/src/index.ts Show resolved Hide resolved
packages/typescript/src/index.ts Outdated Show resolved Hide resolved
@RebeccaStevens RebeccaStevens requested review from NotWoods and removed request for shellscape August 31, 2022 01:39
@perrin4869
Copy link
Contributor

For what it's worth, I think #1194 already adds support for this?

@RebeccaStevens
Copy link
Author

Yeah, that's true. It would be good if we can get one of these merged soon as currently libraries using the .d.mts extension cannot be used in rollup build environment.

@perrin4869
Copy link
Contributor

I think #1194 is more complete and merging this one will only make it harder to resolve conflicts
@NotWoods any chance you can review that PR?
I can resolve the new conflicts but i don't want to keep resolving conflicts indefinitely...

@shellscape
Copy link
Collaborator

Thanks for the work on this PR. #1194 supersedes this PR and we've just merged that.

@shellscape shellscape closed this Sep 6, 2022
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.

None yet

4 participants