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

added php 7.2 && nightly in travis #135

Closed
wants to merge 33 commits into from

Conversation

kokspflanze
Copy link
Contributor

added php 7.2 && nightly in travis
removed hhvm in travis

.travis.yml Outdated
- php: 7.2
env:
- DEPS=latest
- php: nightly
Copy link
Member

Choose a reason for hiding this comment

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

Not sure if we need both - PHP 7.2 and nightly.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

nightly is php7.3

Copy link
Member

Choose a reason for hiding this comment

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

Great, thanks! We should then update maintainers template probably ;-)

@froschdesign
Copy link
Member

LGTM 👍

.travis.yml Outdated
@@ -45,6 +30,8 @@ matrix:
- php: 7
env:
- DEPS=locked
- CHECK_CS=true
Copy link
Member

Choose a reason for hiding this comment

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

Expected variable is CS_CHECK, please see line 84

.travis.yml Outdated
irc: "irc.freenode.org#zftalk.dev"
email: false
- php: 7.2
- php: nightly
Copy link
Member

Choose a reason for hiding this comment

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

Too early... Can be removed, see: zendframework/zend-expressive-hal#9

.travis.yml Outdated
notifications:
irc: "irc.freenode.org#zftalk.dev"
email: false
- php: 7.2
Copy link
Contributor

Choose a reason for hiding this comment

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

Now shouldn't be allowed to fail, right? :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, its a pr from august 2017
i dont want to update it each month and wait more and more

@weierophinney
Copy link
Member

Let's do the following:

  • Allow 7.2 to pass
  • Remove nightly (we can re-enable when either an alpha or beta is first released for 7.3)

remove php 72 from allow fail
@kokspflanze
Copy link
Contributor Author

nightly was already removed with 94e9ecc

@weierophinney weierophinney added this to the 2.9.1 milestone Jan 16, 2018
…k-navigation

Added navigation helpers in DocBlock of navigation proxy helper for code completion

before_install:
- if [[ $TEST_COVERAGE != 'true' ]]; then phpenv config-rm xdebug.ini || return 0 ; fi
- composer self-update
- travis_retry composer self-update
Copy link
Member

Choose a reason for hiding this comment

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

We don't need it anymore

@weierophinney weierophinney modified the milestones: 2.9.1, 2.10.0 Jan 17, 2018
weierophinney added a commit that referenced this pull request Jan 17, 2018
weierophinney added a commit that referenced this pull request Jan 17, 2018
@weierophinney
Copy link
Member

Thanks, @kokspflanze; merged to develop for release with 2.10.0.

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.

8 participants