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.

None yet

4 participants