-
Notifications
You must be signed in to change notification settings - Fork 420
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
Update npm modules #103
Update npm modules #103
Conversation
Fixes jquerygh-96 Closes jquerygh-103
Looks good, though it'd be nice to have coverage state tracked in CI. |
it's currently only reported on Travis, but we can add it later with any other tool like Coveralls or manually. Can we do this on a follow up PR? |
Fixes jquerygh-96 Closes jquerygh-103
the branch is rebased to remove conflicts, no changes were made in the contents of this PR. |
Apparently browserstack is failing for PRs but still passing on master. ... |
This looks good then once CI is fixed for PRs. |
Fixes jquerygh-96 Closes jquerygh-103
I removed the coverage part as grunt-qunit-istanbul is triggering a failure on Mac OS Sierra, I need to update this module. I'll fix the CI run and get this up to date |
No description provided.