-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[appveyor] Run the test suite on PHP 7.1 #22560
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
Conversation
62c0a5d
to
3e4f1c5
Compare
Strange failure on a previously (missconfigured) build: |
FYI, I wrote a Powershell script to automate the install of PHP on AppVeyor (as chocolatey was super buggy/inconsistent for me). I put the script out here: https://github.com/ChadSikorra/ps-install-php Example usage here: https://github.com/ldaptools/ldaptools-bundle/blob/master/appveyor.yml Feel free to copy the script if you feel like incorporating it. It's not the cleanest Powershell script, but it does the job. |
3e4f1c5
to
f4983b6
Compare
now green |
d9ef3d8
to
d83a7f1
Compare
d83a7f1
to
a537d6c
Compare
This PR was merged into the 2.7 branch. Discussion ---------- [appveyor] Run the test suite on PHP 7.1 | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - Commits ------- a537d6c [appveyor] Run the test suite on PHP 7.1
Shouldn't the AppVeyor build essentially mirror the Travis build? I'd think testing such a large code base against the same versions of PHP on Windows as is done on Linux in Travis would be the target? |
Nope, it's not: we use appveyor to test different conditions than on Travis on purpose, in order to exercice the code base in more diverse situations (different extensions loaded + lowest supported version of PHP) |
@ChadSikorra Appveyor is not good at running multiple jobs with different PHP versions in parallel (cache is not keyed by PHP version, invalidating the cache for each job). This is why we restricted ourselves to a single job. |
Uh oh!
There was an error while loading. Please reload this page.