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 benchmarks from npm test #52

Closed
wants to merge 1 commit into from
Closed

remove benchmarks from npm test #52

wants to merge 1 commit into from

Conversation

jonathanong
Copy link
Contributor

this makes running npm test unnecessarily long

this makes running `npm test` unnecessarily long
@lovell
Copy link
Owner

lovell commented Jun 4, 2014

Thanks Jonathan. If it's OK with you, I'd like to add a few more functional tests before doing this as the perf tests are currently providing some of that coverage (sadface, I know).

@jonathanong
Copy link
Contributor Author

sounds good to me! would also be nice if you used a test framework like mocha so you could have pretty outputs

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

Successfully merging this pull request may close these issues.

None yet

2 participants