Thanks to visit codestin.com
Credit goes to github.com

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

Support PHPUnit 3.8+ compatibility#5781

Merged
Maks3w merged 2 commits into
zendframework:masterfrom
JoelMarcey:master
Feb 7, 2014
Merged

Support PHPUnit 3.8+ compatibility#5781
Maks3w merged 2 commits into
zendframework:masterfrom
JoelMarcey:master

Conversation

@JoelMarcey
Copy link
Copy Markdown
Contributor

PHPUnit 3.8+ adds a method to its PHPUnit_Framework_TestListener interface called addRiskyTest(). Need to stub it out to avoid "must implement this interface method" fatals when using 3.8+.

PHPUnit 3.8+ adds a method to its PHPUnit_Framework_TestListener interface called addRiskyTest(). Need to stub it out to avoid "must implement this interface method" fatals when using 3.8+.
@Maks3w
Copy link
Copy Markdown
Member

Maks3w commented Feb 4, 2014

Please remove the trailing spaces

@JoelMarcey
Copy link
Copy Markdown
Contributor Author

I am sorry -- what trailing spaces?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@JoelMarcey
Copy link
Copy Markdown
Contributor Author

I believe I fixed this with c29404d

@JoelMarcey
Copy link
Copy Markdown
Contributor Author

friendly ping -- how often do pull requests get merged, out of curiosity?

@Ocramius
Copy link
Copy Markdown
Member

Ocramius commented Feb 7, 2014

@JoelMarcey AFAIK, everyone conferencing right now.

@JoelMarcey
Copy link
Copy Markdown
Contributor Author

@Ocramius Ahh. Thanks. I am just looking forward to removing using my fork and using the official branch in our test runner :)

@ghost ghost assigned Maks3w Feb 7, 2014
Maks3w added a commit that referenced this pull request Feb 7, 2014
Maks3w added a commit that referenced this pull request Feb 7, 2014
Maks3w added a commit that referenced this pull request Feb 7, 2014
@Maks3w Maks3w merged commit c29404d into zendframework:master Feb 7, 2014
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