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

Add fast-test script to package.json that doesn't run code-coverage. #786

Closed
wants to merge 1 commit into from

Conversation

nlutsenko
Copy link
Contributor

Waaaaay faster local development.
@drew-gross @andrewimm - how can I make it work with posttest and have an ability to pass arguments into it?

@flovilmart
Copy link
Contributor

Did you try:

; npm run posttest -- $?

@flovilmart
Copy link
Contributor

You need -- between the npm run (cmd) -- args to pass arguments.

@nlutsenko
Copy link
Contributor Author

That's great, how do I reference them inside the script?
As you can see - k have pretest and post test inside of the same script, as those won't get picked up.

@flovilmart
Copy link
Contributor

would be better to run coverage only on TRAVIS=1 OR COVERAGE=1 what do you think?

@nlutsenko
Copy link
Contributor Author

Continuing in #807

@nlutsenko nlutsenko closed this Mar 3, 2016
@nlutsenko nlutsenko deleted the nlutsenko.fasttest branch March 3, 2016 17:57
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.

4 participants