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

vinxi build --config <path> broken in 0.4.3 #388

Open
paularmstrong opened this issue Oct 2, 2024 · 4 comments
Open

vinxi build --config <path> broken in 0.4.3 #388

paularmstrong opened this issue Oct 2, 2024 · 4 comments

Comments

@paularmstrong
Copy link

This worked fine in 0.4.1:

vinxi build --config ./config/app.config.ts

After upgrading to 0.4.3:

vinxi build --config ./config/app.config.ts

vinxi v0.4.3


⚙  Building your app...
vinxi v0.4.3

 ERROR  Couldn't load app                                                                 8:37:10 AM

  at Object.run (web/node_modules/vinxi/bin/cli.mjs:231:12)
  at async runCommand (web/node_modules/citty/dist/index.mjs:316:16)
  at async runCommand (web/node_modules/citty/dist/index.mjs:307:11)
  at async runMain (web/node_modules/citty/dist/index.mjs:445:7)



 ERROR  Couldn't load app                                                                 8:37:10 AM


 ERROR  Exited with code: 1                                                               8:37:10 AM

  at CommandChild.pipedStdoutBuffer (web/node_modules/dax-sh/esm/mod.js:9392:19)
  at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Vinxi will only build if the config is in the same directory as the CWD.

@nksaraf
Copy link
Owner

nksaraf commented Oct 9, 2024

Okay sounds like a bug .. I'll try to take a look in this week

@remi-espie
Copy link

Hello, I'm facing the same issue and no updates here. Does a workaround have been found ? Thanks

@hennessyevan
Copy link

I'm not sure what changed, 0.5.0 doesn't work either

@nksaraf
Copy link
Owner

nksaraf commented Dec 8, 2024

Heyy my bad the fix for this is not in, if anyone's got the time, the fix should be simple, a PR would be very welcome. Otherwise I'll try to get to it this week

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants