-
Notifications
You must be signed in to change notification settings - Fork 394
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
Nest CLI Assets Property Fails to Copy Files on Windows Due to Path Format #2397
Comments
please share the output of |
|
This is a valid workaround for the development environment. Adding
|
Would you like to create a PR for this issue? 🙏 |
@kamilmysliwiec, Sure, I can do that. |
@eranbetzalel had you time to create a PR? This is quite bad issue for me |
@kamilmysliwiec Is this a good "first issue"? I'd like to create a PR to fix this |
Feel free to do so @CustomEntity |
Fixed in 10.4.0 |
Thank you @CustomEntity. |
Is there an existing issue for this?
Current behavior
When using the assets property in nest-cli.json, the file copying returns 0 files on Windows. This seems to be caused by
path.join
converting paths to Windows-style backslashes, which the glob package does not recognize.I've debugged the code and saw this specific behavior in the copyAssets function.
Minimum reproduction code
Not available, issue observable in the behavior of nest-cli
Steps to reproduce
Define assets in nest-cli.json using glob patterns.
Run a build or start command that triggers assets copying.
Observe that no files are copied.
Expected behavior
The assets defined in nest-cli.json should be correctly copied regardless of the OS path style.
Package version
10.2.1
NestJS version
10.2.8
Node.js version
18.17.1
In which operating systems have you tested?
Other
No response
The text was updated successfully, but these errors were encountered: