-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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]: yarn install:server
does not work
#4377
Comments
Thanks for raising. Looks like an annoying yarn issue - it doesn't let you run scripts unless you've done an install 🤦♂ . Funnily enough, npm seems to work:
I'll have a look. We may have to remove the script. |
I have seen some discussions about this being an issue with recent versions of yarn. I guess as long as we can run |
Issue reported here: actualbudget/actual#4377 https://deploy-preview-645.www.actualbudget.org/docs/install/local It turns out that yarn doesn't let you run scripts without first installing dependencies, so users will have to type it in the install manually. The other alternative is to run the script with npm but that would add confusion. I think yarn workspace focus is the best option to pick but here was the options: - Instruct users to ```yarn install``` (which would install all deps) - 1.07GB - Instruct users to ```yarn workspaces focus @actual-app/sync-server --production``` (which is a long command) - 60.6MB - Create a new shell script to run the command ``` ./install-server.sh``` In future we might want to consider packaging the server up into an executable. That way the user wont have to install node/deal with any of this stuff. Here's an example packager - i've never used it: https://github.com/nexe/nexe
Have updated the docs: actualbudget/docs#645 The change is to run the command without the script for now. Closing this. |
Verified issue does not already exist?
What happened?
The new script
yarn install:server
does not work out of the box butyarn workspaces focus actual-sync --production
does.As per the documentation (https://actualbudget.org/docs/install/local#installing-actual) it seems that it should work.
I tried with several version of node/npm with the same result, above example uses v18.16.0 as specified in
.nvmrc
I looked a bit online but most answer suggest to run
yarn install
first, that kind of lose the purpose ofyarn install:server
that is supposed to perform a "light" install, right?How can we reproduce the issue?
Where are you hosting Actual?
Locally via Yarn
What browsers are you seeing the problem on?
Other
Operating System
Linux
The text was updated successfully, but these errors were encountered: