Skip to content
This repository has been archived by the owner on Jan 8, 2019. It is now read-only.

Commit

Permalink
Oh, right. Silly me. Composer doesn't work in PHP 5.2.
Browse files Browse the repository at this point in the history
  • Loading branch information
ramsey committed Mar 20, 2015
1 parent a045f5a commit f8e52eb
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
language: php

php:
- 5.2
- 5.3
- 5.4
- 5.5
Expand Down

4 comments on commit f8e52eb

@Rarst
Copy link

@Rarst Rarst commented on f8e52eb Mar 26, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can switch to newer PHP for Composer, than back to 5.2 for tests or whatever:

before_script:
- phpenv local 5.6
- composer selfupdate --no-interaction
- composer install --no-interaction
- phpenv local --unset

@ramsey
Copy link
Owner Author

@ramsey ramsey commented on f8e52eb Mar 26, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While I can do this, I worry that this will cause Composer to pull down PHPUnit, CodeSniffer, etc. that use features of 5.3+ but won't work in 5.2.

@Rarst
Copy link

@Rarst Rarst commented on f8e52eb Mar 26, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just saw in passing and commented since I do this in work projects. CodeSniffer works on 5.2 (unless you got third party sniffs demanding newer), for PHPUnit Travis runs older 3.6 version.

@ramsey
Copy link
Owner Author

@ramsey ramsey commented on f8e52eb Mar 26, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm pulling down the newer versions of both in my composer.json require-dev section, and I have .travis.yml config set to run them from my vendor/bin/, so I know they won't run on 5.2. :-)

Please sign in to comment.