Skip to content
This repository has been archived by the owner on Aug 11, 2022. It is now read-only.

Add Configuration to NPM for Travis CI Builds #4892

Closed
wants to merge 1 commit into from

Conversation

bcoe
Copy link
Contributor

@bcoe bcoe commented Mar 17, 2014

I've added configuration that allows NPM to be built with Travis CI. Mainly, this required the addition of a .travis.yml file to the root directory of the repo.

A few notes:

  • test/tap/lifecycle-signal.js fails on Travis, but runs locally. I could reproduce the problem locally by running: sudo node -e 'process.kill(process.pid,\"SIGSEGV\")' as a preinstall step, rather than node -e 'process.kill(process.pid,\"SIGSEGV\")'. Makes me think it has something to do with the permissions with which a travis agent executes a child-process. I could not get to the bottom of the problem, and have disabled this test for Travis (for the time being).
  • test/tap/publish-config.js hangs in node v0.8.26, I can reproduce this locally. For the time being, I've configured travis to run with a 0.11 agent, and a 0.10 agent.
  • You can see builds passing here: https://travis-ci.org/bcoe/npm
  • I've added a badge to the README, that indicates build status:

screen shot 2014-03-17 at 1 22 57 pm

Let me know if you have any feedback 馃憤, for instance, a strong objection to using Travis CI.

@bcoe
Copy link
Contributor Author

bcoe commented Mar 17, 2014

Ahh, I see that we did have Travis in, but were running into build issues?

a399e65...6aa633e

It seems as though builds are more stable now, with node v0.10+. Worth giving it another shot perhaps?

@bcoe
Copy link
Contributor Author

bcoe commented Mar 17, 2014

This pull should fix the test which is currently being skipped on Travis:

#4663

@bcoe
Copy link
Contributor Author

bcoe commented Mar 17, 2014

I'm closing this pull, based on IRC discussion -- will work to eliminate NPM's external dependencies, then we can move towards CI.

@bcoe bcoe closed this Mar 17, 2014
@othiym23 othiym23 added the bug label Sep 24, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants