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

Dorelys-w1-Node.js #11

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dorelysrod
Copy link

No description provided.

@sarlam sarlam self-assigned this Sep 30, 2024
Copy link

@sarlam sarlam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The service itself looks really great, nice work here

One thing is npm run start do not work right now because of nodemon, I made a comment on how to fix it and make it work :)

assignments/hackyourtemperature/package.json Outdated Show resolved Hide resolved
"version": "1.0.0",
"main": "index.js",
"type": "module",
"scripts": { "start": "nodemon server.js"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for this to work, it requires nodemon to be available, if you do not know what nodemon is you can either read upon it here or change this script to: node server.js

If you want to have nodemon enabled then you need to first run:

npm install --save-dev nodemon

This should update your package.json

@sarlam sarlam added the Approved label Oct 4, 2024
Co-authored-by: Paul Jarleton <5551066+sarlam@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants