-
Notifications
You must be signed in to change notification settings - Fork 161
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
Add -watch option to dev task #215
Conversation
Wouldn't it makes sense to add the `-watch` option to the dev task?
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.
Makes senses for me. LGTM
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.
lgtm
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.
Ci is failing due this change
https://travis-ci.org/fastify/fastify-cli/jobs/634795649?utm_medium=notification&utm_source=github_status
Would you mind to fix it? 😁
The pull request fastify#215 breaks the tests, because the test dev task command is no longer equal with the dev task which is defined in `package.json`. See also: https://travis-ci.org/fastify/fastify-cli/jobs/634795649?utm_medium=notification&utm_source=github_status
What a great first pull request of mine ;-) |
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.
lgtm
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.
A PR with some failing tests and then fixed is better than zero PR!
Thanks for your contribution 💪🏻
Wouldn't it makes sense to add the
-watch
option to the dev task?