-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
feat(cli): lint watch mode #11983
feat(cli): lint watch mode #11983
Conversation
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.
Can you add integration tests to make sure it works fine? You can refer to the existing examples of watcher tests in https:/denoland/deno/blob/d236f432b86de55c6006778b0c68fe60b6419069/cli/tests/integration/watcher_tests.rs
sure, I'll definitely do that |
@magurotuna Added tests, please re-review it :) |
Will change these tonight, it’s Monday morning here, gotta go to work |
When will this gets to merge |
It is tagged for 1.5, merge window for that opens in about two weeks 🙂 |
Signed-off-by: CGQAQ <[email protected]>
Signed-off-by: CGQAQ <[email protected]>
Signed-off-by: CGQAQ <[email protected]>
Signed-off-by: CGQAQ <[email protected]>
Signed-off-by: CGQAQ <[email protected]>
Signed-off-by: CGQAQ <[email protected]>
Signed-off-by: CGQAQ <[email protected]>
Signed-off-by: CGQAQ <[email protected]>
c0875de
to
856ea3e
Compare
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, thanks @CGQAQ
Signed-off-by: CGQAQ [email protected]
fix #11132