Skip to content

Conversation

HendrikRoehm
Copy link
Contributor

In commit c216c0a, the task unittestWatch was removed. However, as of now, the watch task links to it, when executed with the test flag. As watching the unittest is possible with gulp unittest --watch, this code is not needed anymore and thus removed.

In commit c216c0a, the task unittestWatch was removed. However, the watch task links to it, when executed with the test flag. As watching the unittest is possible with `gulp unittest --watch`, this code is not needed anymore and thus removed.
@benmccann
Copy link
Contributor

I usually run gulp test --watch. Does that still work after this change?

Copy link
Member

@simonbrunel simonbrunel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @HendrikRoehm

@simonbrunel simonbrunel added this to the Version 2.8 milestone Nov 1, 2018
@simonbrunel simonbrunel merged commit 820d289 into chartjs:master Nov 1, 2018
@HendrikRoehm
Copy link
Contributor Author

Yes, gulp test --watch works. The removed code was for a different watch approach from the past.

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

Successfully merging this pull request may close these issues.

3 participants