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

[Bug Report]: v2 config command logger should use short path name in windows #1859

Closed
ErKeLost opened this issue Oct 22, 2024 · 4 comments · Fixed by #1944 · May be fixed by #1835
Closed

[Bug Report]: v2 config command logger should use short path name in windows #1859

ErKeLost opened this issue Oct 22, 2024 · 4 comments · Fixed by #1944 · May be fixed by #1835
Assignees
Labels
bug Something isn't working platform: windows pr welcome Welcome contributions for newcomers v2.0 v2.0 version

Comments

@ErKeLost
Copy link
Member

ErKeLost commented Oct 22, 2024

Steps to reproduce

version

"@farmfe/cli": "2.0.0-nightly-20241022101407",
"@farmfe/core": "2.0.0-nightly-20241022041556",
"@farmfe/plugin-react": "2.0.0-nightly-20241022124925",

use example in windows command line should be use shortname

image

Reproduce link

No response

What is actually happening?

absolute path now

✅ farm.config.ts
❌ /xx/farm/examples/react/farm.config.ts

System Info

windows
@ErKeLost ErKeLost added bug Something isn't working v2.0 v2.0 version platform: windows labels Oct 22, 2024
@remloyal
Copy link

Execute Farm Clean
report errors:

[ Farm ] Error: Error cleaning cache in undefined: The "path" argument must be of type string or an instance of Buffer or URL. Received undefined

version:

@farmfe/core: 2.0.0-nightly-20241023020505

@ErKeLost
Copy link
Member Author

@remloyal This should be another bug that should be related to parsing cache config #1860

@ErKeLost ErKeLost added the pr welcome Welcome contributions for newcomers label Oct 23, 2024
@K0nnyaku
Copy link

Hi there, I am a new comer, can I try to solve this issue?

@fu050409
Copy link
Member

Sure, feel free to go ahead!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working platform: windows pr welcome Welcome contributions for newcomers v2.0 v2.0 version
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants