Skip to content
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

Remove dead and broken code from gulpfile #5794

Merged
merged 1 commit into from Nov 1, 2018

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.

None yet

3 participants