Skip to content
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

add missing use statement #20163

Merged
merged 1 commit into from
Oct 5, 2016
Merged

add missing use statement #20163

merged 1 commit into from
Oct 5, 2016

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Oct 5, 2016

Q A
Branch? 2.7
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets 996c1ee#commitcomment-19292581
License MIT
Doc PR

@xabbuh xabbuh changed the base branch from master to 2.7 October 5, 2016 16:02
@stof
Copy link
Member

stof commented Oct 5, 2016

@xabbuh you need to rebase your branch so that it starts from 2.7 too. Otherwise, you are asking us to merge the master branch codebase into 2.7, which won't happen

@xabbuh
Copy link
Member Author

xabbuh commented Oct 5, 2016

@stof already done (I switched the branch here as I read @fabpot's command in #20116 (comment) too late)

@dunglas
Copy link
Member

dunglas commented Oct 5, 2016

👍

1 similar comment
@stof
Copy link
Member

stof commented Oct 5, 2016

👍

@fabpot
Copy link
Member

fabpot commented Oct 5, 2016

Thank you @xabbuh.

@fabpot fabpot merged commit 32451b1 into symfony:2.7 Oct 5, 2016
fabpot added a commit that referenced this pull request Oct 5, 2016
This PR was merged into the 2.7 branch.

Discussion
----------

add missing use statement

| Q             | A
| ------------- | ---
| Branch?       | 2.7
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | 996c1ee#commitcomment-19292581
| License       | MIT
| Doc PR        |

Commits
-------

32451b1 add missing use statement
@xabbuh xabbuh deleted the pr-20116 branch October 5, 2016 18:19
This was referenced Oct 27, 2016
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