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

Checkpoint Dyrolyn Allen #138

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

Conversation

joeyd2212
Copy link

2
3

@0xZakk
Copy link

0xZakk commented Jul 30, 2019

@joeyd2212 You got 2 out of 18 tests passing, below the 60% necessary for completion. That's probably not the score you were hoping for. Why do you think you got this score? What would you do differently next time (i.e. in the next unit)? What can you learn from this?

This assignment is incomplete, for now. Can you go through and make it so all 18 tests are passing? Once you do, we'll mark it as complete.

Thank you!

@joeyd2212
Copy link
Author

Hello,
No that was not the score i was hoping for. I got the previous score because 1. I had nodemon running in the background which, I did not know would interfere with the npm test, as I'm still trying to understand what needs to be running and when. Also, a lot of syntax error and path errors. Closer attention to minor errors.

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 this pull request may close these issues.

2 participants