8000 Fix typo in test method name by kubk · Pull Request #26602 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

Fix typo in test method name #26602

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

Closed
wants to merge 1 commit into from
Closed

Fix typo in test method name #26602

wants to merge 1 commit into from

Conversation

kubk
Copy link
@kubk kubk commented Mar 19, 2018
Q A
Branch? master
New feature? no
BC breaks? no
Deprecations? no
License MIT
Doc PR no

Method testCachWithInvalidCallback replaced with testCacheWithInvalidCallback

@nicolas-grekas
Copy link
Member

Good catch, thanks @kubk.

nicolas-grekas added a commit that referenced this pull request Mar 19, 2018
This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #26602).

Discussion
----------

Fix typo in test method name

| Q             | A
| ------------- | ---
| Branch?       | master
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| License       | MIT
| Doc PR        | no

Method test**Cach**WithInvalidCallback replaced with test**Cache**WithInvalidCallback

<!--
Write a short README entry for your feature/bugfix here (replace this comment block.)
This will help people understand your PR and can be used as a start of the Doc PR.
Additionally:
 - Bug fixes must be submitted against the lowest branch where they apply
   (lowest branches are regularly merged to upper ones so they get the fixes too).
 - Features and deprecations must be submitted against the master branch.
-->

Commits
-------

cc38c6b Fix typo in test method name
@xabbuh xabbuh added this to the 2.7 milestone Apr 20, 2018
@xabbuh xabbuh added the Config label Apr 20, 2018
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.

4 participants
0