Skip to content

Completed Node.js Assignments #11

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

samira313
Copy link

No description provided.

@eidosam eidosam self-assigned this Feb 19, 2025
Copy link

@eidosam eidosam left a comment

Choose a reason for hiding this comment

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

Excellent work, Samira! You've done a great job on this assignment.
The code is clean and in a good structure, the usage of ES modules is great, and defining the port as a constant is a good practice.

Try to add a brief summary of the changes and the purpose of the PR in the description, this would be very helpful for reviewers.

Keep up the good work.

"start": "node server.js"
},
"type": "module",
"author": "",
Copy link

Choose a reason for hiding this comment

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

Although it's optional, including the author's name and email address in the format Name <email-address> in the author field is a good practice.

@samira313
Copy link
Author

samira313 commented Feb 22, 2025 via email

@eidosam eidosam added the Week 1 Week 1 assignment label Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Week 1 Week 1 assignment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants