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

_official-realtime-app can't be run #106

Closed
SylRob opened this issue Dec 19, 2022 · 11 comments · Fixed by #132
Closed

_official-realtime-app can't be run #106

SylRob opened this issue Dec 19, 2022 · 11 comments · Fixed by #132

Comments

@SylRob
Copy link

SylRob commented Dec 19, 2022

Hi, when I run the example I get the error

node: bad option: --watch
ERROR: "dev:server" exited with 9.

my node version is 18.9.1

way to reproduce the error

npm install
npm run dev
@machour
Copy link
Collaborator

machour commented Dec 19, 2022

The example requires Node 19

@MichaelDeBoey
Copy link
Member

@ryanflorence Can we rewrite the code to make it work on Node v14?

@daniserrano7
Copy link

daniserrano7 commented Jan 12, 2023

To make it work, I changed it to

"dev": "concurrently "npm run dev:css" "remix dev""

I use Node.js 16.15.0.

@MichaelDeBoey
Copy link
Member

@SylRob Could you please try again now that we have #126 merged?

@MichaelDeBoey MichaelDeBoey added the needs-response We need a response from the original author about this issue/PR label Jan 13, 2023
@SylRob
Copy link
Author

SylRob commented Jan 16, 2023

@MichaelDeBoey , nop still same for me.
pulled from main main branch, deleted node_module/, npm install, npm run dev

node: bad option: --watch
ERROR: "dev:server" exited with 9.

@machour
Copy link
Collaborator

machour commented Jan 16, 2023

@SylRob what node version are you using?

@github-actions github-actions bot removed the needs-response We need a response from the original author about this issue/PR label Jan 16, 2023
@machour machour added the needs-response We need a response from the original author about this issue/PR label Jan 16, 2023
@SylRob
Copy link
Author

SylRob commented Jan 16, 2023

@machour still v18.9.1

@machour
Copy link
Collaborator

machour commented Jan 16, 2023

The errors says:

node: bad option: --watch

--watch is used in the package.json's commands.

That option was introduced in v18.11.0, so it's not compatible with your node version.

So you can either update your node version to finally be able to make this example work for you, or you can change the commands as @daniserrano7 did to make it work on your current version.

If you do the later, please open a pull requests with your changes so that we can merge it and have this work for node < 18.11

@SylRob
Copy link
Author

SylRob commented Jan 16, 2023

I see,
then I will update my node version to the latest recommended one.
(tomorrow, I will open a PR to update the package.json of the _official-realtime-app to change the node engine requested version)

thank you for looking into it and help me with that

@machour
Copy link
Collaborator

machour commented Jan 16, 2023

No worries.
But I rather have the example made compatible with more node versions, instead of requiring a specific one

@github-actions github-actions bot removed the needs-response We need a response from the original author about this issue/PR label Jan 16, 2023
@SylRob
Copy link
Author

SylRob commented Jan 16, 2023

Yes that is true, but if the targeted version is the node v14, I will not be able to test it, since I can only propose a (possible) working and tested version using @daniserrano7 proposal and my 18.9 ...

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

Successfully merging a pull request may close this issue.

4 participants