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.

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

Faster feedback from Travis #3091

Merged
merged 4 commits into from
Jul 31, 2015
Merged

Conversation

pamil
Copy link
Contributor

@pamil pamil commented Jul 29, 2015

  • First feedback after 19 minutes compared to 32 minutes before
  • Changed order to see most errors after 4 jobs, newest PHP 5.6 is first, then PHP 5.3 with oldest possible libraries is tested
  • For now, PHP7 tests are executed at the end and build is passing just after last required test passed
  • More verbose output, shows at what time and what command was executed, no more counting commands order
  • Command suites are written clearer and don't contain any Travis logic
  • Removed commented out lines to make Travis files clearer

@pamil pamil force-pushed the travis-faster-feedback branch 2 times, most recently from 0456ba3 to 5c67512 Compare July 29, 2015 12:46
pjedrzejewski pushed a commit that referenced this pull request Jul 31, 2015
@pjedrzejewski pjedrzejewski merged commit cc7f198 into Sylius:master Jul 31, 2015
@pjedrzejewski
Copy link
Member

Thank you Kamil, awesome work!

@pamil pamil deleted the travis-faster-feedback branch August 5, 2015 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants