-
Notifications
You must be signed in to change notification settings - Fork 6
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
added some legendary quotes #4
Conversation
src/index.js
Outdated
'Writing tests takes x3 time than writing actual code.', | ||
'We need you to work on features. Not performance.', | ||
'Please do not worry a lot about quality. This is a prototype.', | ||
'Hurry up. This is just a proof of concept.', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this one is similar to the one above. Otherwise its all good with me. @jmike ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, @cherouvim line #15 is quite similar to line #14.
Would you agree to remove it from the PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome stuff. A couple of comments on my side.
src/index.js
Outdated
'Writing tests takes x3 time than writing actual code.', | ||
'We need you to work on features. Not performance.', | ||
'Please do not worry a lot about quality. This is a prototype.', | ||
'Hurry up. This is just a proof of concept.', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, @cherouvim line #15 is quite similar to line #14.
Would you agree to remove it from the PR?
src/index.js
Outdated
'We need you to work on features. Not performance.', | ||
'Please do not worry a lot about quality. This is a prototype.', | ||
'Hurry up. This is just a proof of concept.', | ||
'Can we worry about security at a later phase?' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cherouvim suggestion to simplify: "Can we worry about security later?"
What do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requested changes done.
src/index.js
Outdated
'Writing tests takes x3 time than writing actual code.', | ||
'We need you to work on features. Not performance.', | ||
'Please do not worry a lot about quality. This is a prototype.', | ||
'Hurry up. This is just a proof of concept.', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
src/index.js
Outdated
'We need you to work on features. Not performance.', | ||
'Please do not worry a lot about quality. This is a prototype.', | ||
'Hurry up. This is just a proof of concept.', | ||
'Can we worry about security at a later phase?' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
Accepted and merged manually. |
No description provided.