-
Notifications
You must be signed in to change notification settings - Fork 41
Updated nikic/php-parser to 4.x. #147
base: master
Are you sure you want to change the base?
Updated nikic/php-parser to 4.x. #147
Conversation
c3728a8 to
7c69a23
Compare
7c69a23 to
4e1e076
Compare
|
Would be great to get this updated, the dependency to |
|
I don't really know why travis is not executing tests in this branch atm but the tests are failing. I also created a PR some time ago to get to next major version but never really finished it |
| } | ||
| ], | ||
| "require": { | ||
| "php": ">=5.5.9 || ^7.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since tests no longer run on <=7.1, the minimum PHP version should be bumped as well.
|
|
||
| matrix: | ||
| include: | ||
| - php: 5.5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
7.2 and 7.3 should be added to the matrix.
|
Duplicate of #145 |
No description provided.