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

unused parameters $sm for plugins factory #6393

Closed
wants to merge 3 commits into from
Closed

unused parameters $sm for plugins factory #6393

wants to merge 3 commits into from

Conversation

jmleroux
Copy link
Contributor

No description provided.

@DASPRiD
Copy link
Member

DASPRiD commented Jun 19, 2014

Just for the record: The failing test on PHP 5.6 is not due to this PR.

@jmleroux
Copy link
Contributor Author

Just for the record: The failing test on PHP 5.6 is not due to this PR.

A problem with fractionnal DateTime in PHP 5.6

@Ocramius
Copy link
Member

We shouldn't allow failures in 5.6. Please revert the change on .travis.yml
The PR would be merged anyway.

@jmleroux
Copy link
Contributor Author

Please revert the change on .travis.yml

⏪ done

@ezimuel ezimuel added this to the 2.3.2 milestone Aug 6, 2014
@Ocramius Ocramius closed this in 9b75279 Aug 6, 2014
Ocramius added a commit that referenced this pull request Aug 6, 2014
@Ocramius
Copy link
Member

Ocramius commented Aug 6, 2014

@jmleroux I removed the commits related to .travis.yml and merged the PR manually. Thanks!

@Ocramius Ocramius self-assigned this Aug 6, 2014
@jmleroux jmleroux deleted the patch-ViewHelperManagerFactory branch September 23, 2014 14:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants