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

TS build: Fix that path alias to TS files stay unresolved. #24054

Conversation

williamvinogradov
Copy link
Contributor

Fixed the issue that the path alias from the tsconfig isn't resolved after the build.

Problem:
E.g. we have the following paths in the tsconfig:

"paths": {
  "@core-utils/*": "./__internal/core/utils/*"
}

And use is in TS module:

import { something } from '@core-utils/something';

Expected behavior (after fix):
After the build we see resolved import in the js module:

var something = require("../../../../core/utils/something");

Current behavior:
Import isn’t resolved:

var something = require("@core-uitls/something");

@williamvinogradov williamvinogradov requested a review from a team March 20, 2023 11:08
@williamvinogradov williamvinogradov self-assigned this Mar 20, 2023
@williamvinogradov williamvinogradov merged commit 63e28db into DevExpress:23_1 Mar 20, 2023
williamvinogradov added a commit to williamvinogradov/DevExtreme that referenced this pull request Mar 20, 2023
…s#24054)

Co-authored-by: Ilya Vinogradov <ilya.vinogradov@devexpress.com>
williamvinogradov added a commit to williamvinogradov/DevExtreme that referenced this pull request Mar 20, 2023
…s#24054)

Co-authored-by: Ilya Vinogradov <ilya.vinogradov@devexpress.com>
williamvinogradov added a commit that referenced this pull request Mar 20, 2023
…24055)

Co-authored-by: Ilya Vinogradov <ilya.vinogradov@devexpress.com>
williamvinogradov added a commit that referenced this pull request Mar 20, 2023
…24056)

Co-authored-by: Ilya Vinogradov <ilya.vinogradov@devexpress.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants