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.

Sign up for GitHub

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

Jump to bottom

[Travis] Yet another build refactoring #4818

Merged
merged 5 commits into from
Apr 18, 2016
Merged

[Travis] Yet another build refactoring #4818

merged 5 commits into from
Apr 18, 2016

Conversation

pamil
Copy link
Contributor

@pamil pamil commented Apr 18, 2016

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets -
License MIT
  • Use container based infrastructure (as Trusty beta does not allow caching)
  • Cache NPM dependencies
  • Application tests for PHP 7.0, 5.6 and 5.5; packages tests for PHP 7.0 and 5.5
  • Extract packages jobs (so 3 -> 5)
  • Faster feedback cycle (using all jobs for one build at one time)
  • Move application phpunit tests to /tests/, other ones are run at package level
  • Use built-in Chromium v37 instead of Chrome (with a legacy, Chromedriver v2.12)

Basically, faster builds! 🎉

@pamil pamil changed the title [Travis] Yet another build refactoring 🎉 [Travis] Yet another build refactoring Apr 18, 2016
This was referenced Apr 18, 2016
Closed
Closed
@pjedrzejewski pjedrzejewski added the Maintenance CI configurations, READMEs, releases, etc. label Apr 18, 2016
@pjedrzejewski pjedrzejewski merged commit 8ea032d into Sylius:master Apr 18, 2016
@pjedrzejewski
Copy link
Member

$buildConfigurationBusFactor++, thanks Kamil!

@pamil pamil deleted the travis/container-with-chromium branch April 28, 2016 09:26
pamil pushed a commit to pamil/Sylius that referenced this pull request May 7, 2019
Merge pull request Sylius#4818 from pamil/travis/container-with-chromium
c90e4b2 
[Travis] Yet another build refactoring
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Maintenance CI configurations, READMEs, releases, etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pamil @pjedrzejewski