Skip to content

Conversation

@HannaInIT
Copy link

No description provided.

@HannaInIT
Copy link
Author

Important clarification:

I couldn't fork the /Node.js-Cohort53 repo because I had already forked its parent repo /Node.js.

So, I did all the work inside my /Node.js fork. Later, I created a copy of the branch, renamed it, and opened a PR for /Node.js-Cohort53 to be sure I’ve completed the homework assignment.

As a result, I now have two branches and two PRs — one for /Node.js and one for /Node.js-Cohort53.

Also, some commits related to prep.exercises were accidentally included - one of them addition of nodemon, which I had installed globally for prep.exercise.

@hacdias hacdias self-assigned this Aug 6, 2025
@hacdias
Copy link

hacdias commented Aug 10, 2025

Hi @HannaInIT ,

You seem to have done the assignment and prep exercise from Week 1, but the practice exercises are missing.

Here's the full instructions: https://github.com/HackYourAssignment/Node.js-Cohort53/blob/main/week1/MAKEME.md

@HannaInIT
Copy link
Author

Hello @hacdias,
Thank you for pointing this out to me. I didn’t know I needed to do them as well. I’ve already started!

@HannaInIT
Copy link
Author

HannaInIT commented Aug 12, 2025

Hello @hacdias, here is a link to branch with practice exercises https://github.com/HannaInIT/Node.js/tree/hanna-dubyna/practice-exercises-w1-node. Please check it at your convinience.

@hacdias
Copy link

hacdias commented Aug 12, 2025

Hey @HannaInIT: They should be in the same branch, so that I can review them in this PR.

@HannaInIT
Copy link
Author

Done:)

Copy link

@hacdias hacdias left a comment

Choose a reason for hiding this comment

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

Exercises for Week 1 look great. Congratulations!

I see you also added some Week 2 exercises. I will look at those once you have made the submission for Week 2.

@HannaInIT
Copy link
Author

Thank you for checking, Henrique!

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