Skip to content

Add Support for Resolving .cjs, .mjs, .cts and .mts Files #1503

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.

Sign up for GitHub

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 15 commits into from
Sep 19, 2022
Merged

Add Support for Resolving .cjs, .mjs, .cts and .mts Files #1503

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
75a4dfd
resolve: Pass implied module kind to typescript
neurolag Sep 17, 2022
8762de4
resolve: Fix detection of declaration files
neurolag Sep 17, 2022
5919999
test: Replace inexistent package
neurolag Sep 18, 2022
a821e26
test: Fix broken docker image
neurolag Sep 18, 2022
62d8033
devcontainer: Update outdated settings
neurolag Sep 18, 2022
be52871
resolve: Allow `.cts`, `.mts`, `.cjs` and `.mjs`
neurolag Sep 18, 2022
402340c
resolve: Fix broken `enhanced-resolve` calls
neurolag Sep 18, 2022
f6ea7b7
resolve: Improve type safety
neurolag Sep 18, 2022
c218be4
interop: Create a separate regex for declarations
neurolag Sep 18, 2022
076e585
types: Remove unnecessary return type
neurolag Sep 18, 2022
8dc552e
test: Add remark on how to regenerate tests
neurolag Sep 18, 2022
85cfc5a
test: Regenerate comparison-tests
neurolag Sep 19, 2022
c415a49
Bump the version number
neurolag Sep 19, 2022
acb60ef
Add changelog entry
neurolag Sep 19, 2022
c35bf1d
doc: Update CHANGELOG.md
neurolag Sep 19, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
resolve: Fix detection of declaration files
  • Loading branch information
@neurolag
neurolag committed Sep 17, 2022
commit 8762de47befab52c0c614e180c02aa9ad373a3e4
2 changes: 1 addition & 1 deletion src/instances.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -682,7 +682,7 @@ export function getInputFileNameFromOutput(
instance: TSInstance,
filePath: string
): string | undefined {
if (filePath.match(tsTsxRegex) && !fileExtensionIs(filePath, '.d.ts')) {
if (filePath.match(tsTsxRegex) && !/\.d\.([cm]?ts|tsx)$/.test(filePath)) {
Copy link
Member

@johnnyreilly johnnyreilly Sep 18, 2022

Choose a reason for hiding this comment

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

Could we give this regex a name and site it in constants.ts?

return undefined;
}
if (instance.solutionBuilderHost) {
Expand Down