Skip to content

[UE5.5] Fixing module resolution within IDEs (#592)#593

Merged
mcottontensor merged 1 commit intoUE5.5from
backport/UE5.5/pr-592
Apr 10, 2025
Merged

[UE5.5] Fixing module resolution within IDEs (#592)#593
mcottontensor merged 1 commit intoUE5.5from
backport/UE5.5/pr-592

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from master to UE5.5:

Questions ?

Please refer to the Backport tool documentation

* Fixing tsconfigs to help IDEs resolve imports.

Without these changes the IDE fails to find some imports and made it
hard for things like intellisense and auto complete etc.
It shouldn't make a difference with actual builds because the
moduleResolution is overridden anyway.

* Explicitly setting CJS projects to node10 moduleResolution.

node10 is the default moduleResolution for cjs projects so this is just
explicitly setting that so that the bundler entry from the base gets
properly overwritten.

(cherry picked from commit f3e0e91)
@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Apr 10, 2025

⚠️ No Changeset found

Latest commit: ba9ec82

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@mcottontensor mcottontensor merged commit e6b2d45 into UE5.5 Apr 10, 2025
2 checks passed
@mcottontensor mcottontensor deleted the backport/UE5.5/pr-592 branch April 10, 2025 02:23
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.

1 participant