Skip to content
This repository was archived by the owner on Jan 29, 2020. It is now read-only.

Update Travis CI config #259

Merged
merged 15 commits into from
Oct 13, 2017
Merged

Update Travis CI config #259

merged 15 commits into from
Oct 13, 2017

Conversation

michalbundyra
Copy link
Member

@michalbundyra michalbundyra commented Jul 13, 2017

  • removes hhvm build
  • adds php 7.1, 7.2 and nightly (7.3-dev) builds (removed as not even alpha has been published)
  • removes irc notifications
  • adds composer.lock to the repository and run tests with lowest/locked/latest dependencies

@michalbundyra
Copy link
Member Author

Any ideas why the last build fails? Is it the issue with cache? Maybe we should add also vendor in travis ci cache.directories?

@kokspflanze
Copy link
Contributor

kokspflanze commented Aug 23, 2017

@webimpress did you mean

Parse error: syntax error, unexpected ':', expecting ';' or '{' in /home/travis/build/zendframework/zend-db/vendor/doctrine/instantiator/src/Doctrine/Instantiator/Instantiator.php on line 95

you install https://github.com/doctrine/instantiator/releases/tag/1.1.0 which require PHP 7.1

thats the problem i mean with the ignore-platform param

@michalbundyra
Copy link
Member Author

@kokspflanze oh, you have right, thanks! The travis configuration here is so different (than others libraries) and probably we can improve it a lot... Definitely we need to add PHP 7.2 support in the library... I need to have another look on it.

It is now handled by zendframework/zfbot
This is not supported in old PHPUnit version.
The argument should be added back when PHPUnit will be updated
in the library.
The library still uses old coding standards so scripts in composer
are now updated to use appropriate tools.
The version of PHPUnit is not detected correctly and it causes issues
when rnning tests with lowest dependencies. This check is no needed
because required version of PHPUnit in the library is version 4.0.0
- HYDRATOR_VERSION='^1.1'
- SERVICE_MANAGER_VERSION='^2.7.5'
- php: 7
- php: 7
Copy link
Contributor

Choose a reason for hiding this comment

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

dont miss to readd php 7 in the matrix

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh, good point, I've missed that. Thanks. Done in c91b08b

@michalbundyra
Copy link
Member Author

Finally green !

/cc @ezimuel @weierophinney @mwillbanks

It's too early to run builds on PHP 7.3-dev.
Alpha has not been published yet.
@akrabat
Copy link
Contributor

akrabat commented Oct 13, 2017

I"m getting:

$ vendor/bin/phpunit -d xdebug.scream=0
PHPUnit 4.8.36 by Sebastian Bergmann and contributors.

.....................SSSSSSSSSSSS...SS.....IIIIIIIIIIIIIIIIII   61 / 1092 (  5%)
SSSS...IIIIIISSSSSSSSSSSS.......SS.....IIIIIIIIIIIIIIIIII....  122 / 1092 ( 11%)
...SSSS.......I.I.I......III..I..............................  183 / 1092 ( 16%)
..........S..SSEE......IIIIISSSSSSSSSSSSS...IIIIIIIIIIIIISS..  244 / 1092 ( 22%)
...IIIIISSSS...IIIIII................................SS......  305 / 1092 ( 27%)
.....................SS......................S...............  366 / 1092 ( 33%)
.......................S.....................................  427 / 1092 ( 39%)
.............................................................  488 / 1092 ( 44%)
.............................S...............................  549 / 1092 ( 50%)
.............................................................  610 / 1092 ( 55%)
.............................................................  671 / 1092 ( 61%)
.............................................................  732 / 1092 ( 67%)
.............................................................  793 / 1092 ( 72%)
.............................................................  854 / 1092 ( 78%)
.............................................................  915 / 1092 ( 83%)
.............................................................  976 / 1092 ( 89%)
............................................................. 1037 / 1092 ( 94%)
.......................................................

Time: 2.21 seconds, Memory: 24.00MB

There were 2 errors:

1) ZendTest\Db\Adapter\Driver\Pgsql\ConnectionTest::testSetCharset
PHPUnit_Framework_Exception: PHP Deprecated:  ini_set(): Use of iconv.input_encoding is deprecated in - on line 70
PHP Stack trace:
PHP   1. {main}() -:0
PHP   2. ini_set() -:70
PHP Deprecated:  ini_set(): Use of iconv.internal_encoding is deprecated in - on line 71
PHP Stack trace:
PHP   1. {main}() -:0
PHP   2. ini_set() -:71
PHP Deprecated:  ini_set(): Use of iconv.output_encoding is deprecated in - on line 72
PHP Stack trace:
PHP   1. {main}() -:0
PHP   2. ini_set() -:72
PHP Deprecated:  ini_set(): Use of mbstring.http_output is deprecated in - on line 98
PHP Stack trace:
PHP   1. {main}() -:0
PHP   2. ini_set() -:98
PHP Deprecated:  ini_set(): Use of mbstring.internal_encoding is deprecated in - on line 100
PHP Stack trace:
PHP   1. {main}() -:0
PHP   2. ini_set() -:100

2) ZendTest\Db\Adapter\Driver\Pgsql\ConnectionTest::testSetInvalidCharset
PHPUnit_Framework_Exception: PHP Deprecated:  ini_set(): Use of iconv.input_encoding is deprecated in - on line 70
PHP Stack trace:
PHP   1. {main}() -:0
PHP   2. ini_set() -:70
PHP Deprecated:  ini_set(): Use of iconv.internal_encoding is deprecated in - on line 71
PHP Stack trace:
PHP   1. {main}() -:0
PHP   2. ini_set() -:71
PHP Deprecated:  ini_set(): Use of iconv.output_encoding is deprecated in - on line 72
PHP Stack trace:
PHP   1. {main}() -:0
PHP   2. ini_set() -:72
PHP Deprecated:  ini_set(): Use of mbstring.http_output is deprecated in - on line 98
PHP Stack trace:
PHP   1. {main}() -:0
PHP   2. ini_set() -:98
PHP Deprecated:  ini_set(): Use of mbstring.internal_encoding is deprecated in - on line 100
PHP Stack trace:
PHP   1. {main}() -:0
PHP   2. ini_set() -:100

FAILURES!
Tests: 1092, Assertions: 1759, Errors: 2, Skipped: 65, Incomplete: 78.

akrabat added a commit that referenced this pull request Oct 13, 2017
akrabat added a commit that referenced this pull request Oct 13, 2017
akrabat added a commit to akrabat/zend-db that referenced this pull request Oct 13, 2017
@akrabat akrabat merged commit 26d9067 into zendframework:master Oct 13, 2017
@michalbundyra michalbundyra deleted the hotfix/travis branch October 20, 2017 22:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants