Skip to content

[HttpKernel] Fix missing psr/cache dev requirement #20894

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 13, 2016

Conversation

chalasr
Copy link
Member

@chalasr chalasr commented Dec 13, 2016

Q A
Branch? master
Bug fix? yes, tests are broken
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets n/a
License MIT
Doc PR n/a

Because #20810 added a test for the Psr6CacheClearer that mocks methods of the CacheItemPoolInterface. It should fix the travis build on master and deps=low

@ogizanagi
Copy link
Contributor

Good catch 👍

To me should solve the failing test, but let's see.

@chalasr chalasr force-pushed the httpkernel/req-psr-cache branch from d0a1479 to bd59d75 Compare December 13, 2016 12:48
@chalasr chalasr changed the title [HttpKernel] Require psr/cache in dev [HttpKernel] Fix missing psr/cache dev requirement Dec 13, 2016
@ogizanagi
Copy link
Contributor

🎉

Status: Reviewed

@xabbuh
Copy link
Member

xabbuh commented Dec 13, 2016

👍

Status: Reviewed

@nicolas-grekas
Copy link
Member

Thank you @chalasr.

@nicolas-grekas nicolas-grekas merged commit bd59d75 into symfony:master Dec 13, 2016
nicolas-grekas added a commit that referenced this pull request Dec 13, 2016
…asr)

This PR was merged into the 3.3-dev branch.

Discussion
----------

[HttpKernel] Fix missing psr/cache dev requirement

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | yes, tests are broken
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

Because #20810 added a test for the `Psr6CacheClearer` that mocks methods of the `CacheItemPoolInterface`. It should fix the travis build on master and deps=low

Commits
-------

bd59d75 [HttpKernel] Require psr/cache in dev
@chalasr chalasr deleted the httpkernel/req-psr-cache branch December 13, 2016 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants