search

dwyl / learn-travis

:sunglasses: A quick Travis CI (Continuous Integration) Tutorial for Node.js developers
MIT License
1.25k stars 352 forks source link

disable npm test #35

Closed shanalikhan closed 6 years ago

shanalikhan commented 6 years ago

what do we need to write to disable the npm test in travis.yml

nelsonic commented 6 years ago

@shanalikhan I don't understand the question. Would you not be better off disabling the build on Travis-CI UI?

shanalikhan commented 6 years ago

This is my config for travis https://github.com/shanalikhan/code-settings-sync/blob/master/.travis.yml its running the npm test after installation. And i dont want to run npm test after installation.

nelsonic commented 6 years ago

@shanalikhan try adding the following line to your .travis.yml file:

script: echo "npm test temporarily disabled"

See: https://travis-ci.org/dwyl/learn-travis/jobs/338953975#L515-L519

nelsonic commented 6 years ago

@shanalikhan did that approach work for you...? 🤔

shanalikhan commented 6 years ago

Yes thanks

Get Outlook for iOShttps://aka.ms/o0ukef

From: Nelson notifications@github.com Sent: Friday, February 9, 2018 7:37:19 PM To: dwyl/learn-travis Cc: Shan Khan; Mention Subject: Re: [dwyl/learn-travis] disable npm test (#35)

@shanalikhanhttps://github.com/shanalikhan did that approach work for you...? 🤔

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/dwyl/learn-travis/issues/35#issuecomment-364450867, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AIXjnJDtY1cKLdaAAq--f4qPDTrtcvyKks5tTFgfgaJpZM4R-KBZ.

redplane commented 3 years ago

I think npm test should be disabled by default. Previously, we had to write npm test command ourselves, but it was OK. Another CI framework, they don't run npm test. This will make travis ci to be consitent.