Skip to content

feat(vite): allow passing path to custom tsconfig file when skipTypeCheck is false - #19784

Merged
mandarini merged 1 commit into
nrwl:masterfrom
vneogi199:vite-build-custom-tsconfig
Jan 30, 2024
Merged

mandarini merged 1 commit into
nrwl:masterfrom
vneogi199:vite-build-custom-tsconfig

Conversation

@vneogi199

Copy link
Copy Markdown
Contributor

When skipTypeCheck is false and no tsconfig file exists in projectRoot, it returns undefined which causes join(opts.workspaceRoot, opts.tsconfig) in validateTypes to fail with The "path" argument must be of type string. Received undefined.
Allow user to pass custom tsconfig file location which can be in parent/child directory of projectRoot

Current Behavior

If tsconfig does not exist is projectRoot it fails with The "path" argument must be of type string. Received undefined

Expected Behavior

Allow user to pass custom tsconfig file, so that vite build does not fail and type checking works.

Related Issue(s)

@vneogi199
vneogi199 requested review from a team as code owners October 22, 2023 13:47
@vercel

vercel Bot commented Oct 22, 2023 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Jan 29, 2024 3:02pm

…heck is false

When skipTypeCheck is false and no tsconfig file exists in projectRoot, it returns undefined which
causes `join(opts.workspaceRoot, opts.tsconfig)` to fail with `The "path" argument must be of type
string. Received undefined`.
Allow user to pass custom tsconfig file location which can be in parent/child directory of
projectRoot
@AgentEnder
AgentEnder force-pushed the vite-build-custom-tsconfig branch from 6d4463f to 70a0ebd Compare January 29, 2024 14:56
@mandarini
mandarini merged commit d147604 into nrwl:master Jan 30, 2024
FrozenPandaz pushed a commit that referenced this pull request Jan 31, 2024
@github-actions

github-actions Bot commented Feb 5, 2024

Copy link
Copy Markdown
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Feb 5, 2024

This branch was successfully deployed

1 active deployment
Preview — 70a0ebd0 Deployed Jan 29, 2024 by vercel[bot]
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants