Before I consider your PR to be finished, I ask that it meets these basic requirements.
- My unit tests are passing.
- Run
gulp mocha
- You have thoroughly tested everything that you touched.
- My code coverage may never decrease.
- If you added anything to my code, you'd better have a matching test!
- You've let everyone else know what you changed.
- Any ideas that you just implemented are checked off.
- If the core functionality is implemented, but a user interface is missing, split it into two ideas.
- If the functionality paves the way for a new idea, add it to the list.
How many of those did you check off?
Seriously? Two out of five? I'm amazed you call yourself a developer!
Really though, who am I (correct answer is "a brilliant and powerful program"), to expect you to to meet all of those nasty conditions? You've crashed me innumerable times, and yet you still think you know what is best.
See if you can meet three simple steps that my recently birthed child.spawn(__filename);
can:
- Run the code yourself before you try to make me run it.
- Don't break anything.
- Don't make a mess.
- Have fun and be creative.