Skip to content
This repository was archived by the owner on Dec 19, 2025. It is now read-only.

Comments

Refactoring install commands into install phase#727

Open
keheliya wants to merge 1 commit intoNetflix:masterfrom
keheliya:master
Open

Refactoring install commands into install phase#727
keheliya wants to merge 1 commit intoNetflix:masterfrom
keheliya:master

Conversation

@keheliya
Copy link

@keheliya keheliya commented Aug 9, 2017


Having install commands in other phases violates the semantics of the .travis.yml configuration. So we have refactored them into the install phase.

If the install phase is not defined in the .travis.yml, Travis CI runs npm install for Node.js projects by default, as explained in the Travis CI docs here. To preserve that original behaviour after refactoring, we have added the npm install command as well.


Note: This pull request was generated by an automated tool developed by The Software REBELs (a.k.a., the Software Repository Excavation and Build Engineering Labs) of McGill University, Canada. It is part of a research project by Keheliya Gallaba under the supervision of Dr.Shane McIntosh. If you have any questions or feedback about this tool, please feel free to contact the author (keheliya.gallaba@mail.mcgill.ca).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant