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 task. #16

Merged
merged 1 commit into from Nov 24, 2015

Conversation

Projects
None yet
2 participants
@denysdovhan
Copy link
Contributor

denysdovhan commented Nov 22, 2015

I think it's very useful thing.

@iamstarkov

This comment has been minimized.

Copy link
Owner

iamstarkov commented Nov 23, 2015

I used to have transpile:dev and you?

@denysdovhan

This comment has been minimized.

Copy link
Contributor

denysdovhan commented Nov 23, 2015

Usualy I use watch task.
BTW: Rise uses this thask too istead of dev or whatever.

@iamstarkov

This comment has been minimized.

Copy link
Owner

iamstarkov commented Nov 23, 2015

can you rebase pull-request? i merged prev one, so now we have conflicts

@iamstarkov

This comment has been minimized.

Copy link
Owner

iamstarkov commented Nov 24, 2015

watch is also good name, can you rebase branch, I will merge it

@denysdovhan

This comment has been minimized.

Copy link
Contributor

denysdovhan commented Nov 24, 2015

of course, wait for a while

@denysdovhan

This comment has been minimized.

Copy link
Contributor

denysdovhan commented Nov 24, 2015

sorry, I'm stupid. accidentally I merged it with master.
Now all right.

iamstarkov added a commit that referenced this pull request Nov 24, 2015

@iamstarkov iamstarkov merged commit c04ca5c into iamstarkov:master Nov 24, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@iamstarkov

This comment has been minimized.

Copy link
Owner

iamstarkov commented Nov 24, 2015

merged

@iamstarkov

This comment has been minimized.

Copy link
Owner

iamstarkov commented Nov 24, 2015

maybe I have some good insights about coverage also

@denysdovhan

This comment has been minimized.

Copy link
Contributor

denysdovhan commented Nov 24, 2015

👍

@denysdovhan

This comment has been minimized.

Copy link
Contributor

denysdovhan commented Nov 24, 2015

I have some good insights about coverage

What do you mean?

@iamstarkov

This comment has been minimized.

Copy link
Owner

iamstarkov commented Nov 24, 2015

@iamstarkov

This comment has been minimized.

Copy link
Owner

iamstarkov commented Nov 24, 2015

it should work

@denysdovhan

This comment has been minimized.

Copy link
Contributor

denysdovhan commented Nov 24, 2015

it seems I've already tried it

@iamstarkov

This comment has been minimized.

Copy link
Owner

iamstarkov commented Nov 24, 2015

didnt work?

@iamstarkov

This comment has been minimized.

Copy link
Owner

iamstarkov commented Nov 24, 2015

it looks like you suggested in coverage pull-request except of switching --require to--compilers`. maybe its the case

@denysdovhan

This comment has been minimized.

Copy link
Contributor

denysdovhan commented Nov 24, 2015

Maybe, not sure.

@denysdovhan denysdovhan deleted the denysdovhan:patch-2 branch Dec 1, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment