Pre-generate the files when running Travis builds for speed #1135

Open
asgrim opened this Issue Dec 6, 2016 · 2 comments

Projects

None yet

2 participants

@asgrim
Member
asgrim commented Dec 6, 2016

If we pre-generate the files for the build instead of building them whilst running PHPUnit, we can see massive performance increases.

Basically:

  • Use php -n to disable any ini files (thus, xdebug does not get loaded) and generate the files
  • THEN run vendor/bin/phpunit (but don't generate the files if they already exist).

If the files don't exist, they should still be generated so the existing behaviour is maintained though!

@asgrim asgrim added this to the Wishlist milestone Dec 6, 2016
@mimmi20
Member
mimmi20 commented Dec 6, 2016

The second phpunit call could be run without xdebug because we call it with the --no-coverage option.

@asgrim
Member
asgrim commented Dec 6, 2016

Yep, but it's the generation of the files (done in the first phpunit run) that takes the time (~10m with xdebug, ~2.5m without)

@mimmi20 mimmi20 modified the milestone: 6020, Wishlist Dec 7, 2016
@mimmi20 mimmi20 self-assigned this Dec 7, 2016
@mimmi20 mimmi20 added a commit to mimmi20/browscap that referenced this issue Dec 7, 2016
@mimmi20 mimmi20 #1135: update travis configuration 4dbd814
@mimmi20 mimmi20 added a commit to mimmi20/browscap that referenced this issue Dec 7, 2016
@mimmi20 mimmi20 #1135: update travis configuration 4685096
@mimmi20 mimmi20 added a commit to mimmi20/browscap that referenced this issue Dec 7, 2016
@mimmi20 mimmi20 #1135: update travis configuration 74e4242
@mimmi20 mimmi20 added a commit to mimmi20/browscap that referenced this issue Dec 7, 2016
@mimmi20 mimmi20 #1135: update travis configuration d5adf7f
@mimmi20 mimmi20 added a commit to mimmi20/browscap that referenced this issue Dec 9, 2016
@mimmi20 mimmi20 #1135: update travis configuration 78b9ede
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment