Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
Already on GitHub? Sign in to your account
Allow build script to reuse pre-installed dependencies #1020
Conversation
|
@harding Duh, obviously! Thanks, I somehow assumed that the build was still using the ./vendor path, and I guessed that I had only thought about the issue without opening a PR in the past... I should have looked more. |
saivann
closed this
Aug 22, 2015
saivann
deleted the
saivann:pre-installed-dependencies branch
Aug 22, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
saivann commentedAug 22, 2015
Currently, our build script needs to download and install dependencies on every build. This probably prolongs build time unnecessarily and introduces a bit more of security risk with no apparent benefit.
This little change allows the build script to stop deleting files listed in .gitignore when fetching recent commits, therefore allowing it to keep and reuse the bundle environment in ./vendor.
Note: This small change is untested.