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(storybook): path on Win for tsconfig location in angular.json #4080
Conversation
2ea516d
to
5d92f64
Compare
tsConfigStorybook: path.join( | ||
options.projectConfig.root, | ||
'.storybook/tsconfig.json' | ||
tsConfig: normalize(path.join(options.projectConfig.root, 'tsconfig.json')), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is required because the project root has \\
in its path?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, there was an issue that Rares mentioned related to these paths on Windows. I'm working on a fix just now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but to be precise, that specific line wouldn't probably need the normalize, I'm gonna remove it there
Nx Cloud ReportCI ran the following commands for commit 1bf5bf4. Click to see the status, the terminal output, and the build insights.
Sent with 💌 from NxCloud. |
5d92f64
to
fcd82e2
Compare
fcd82e2
to
1bf5bf4
Compare
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. |
Current Behavior
Expected Behavior
Related Issue(s)
Fixes #3956