Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[DependencyInjection] Add coverage for invalid Expression in exportParameters #18499

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 11, 2016
Merged

[DependencyInjection] Add coverage for invalid Expression in exportParameters #18499

merged 1 commit into from
Apr 11, 2016

Conversation

JhonnyL
Copy link
Contributor

@JhonnyL JhonnyL commented Apr 10, 2016

Q A
Branch? 2.7
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
License MIT

@JhonnyL JhonnyL changed the title [DependencyInjection] Add coverage for all invalid Expression in ExportParameters [DependencyInjection] Add coverage for all invalid Expression in exportParameters Apr 10, 2016
@JhonnyL JhonnyL mentioned this pull request Apr 10, 2016
Closed
@JhonnyL JhonnyL force-pushed the invalid-expression-in-export-parameters branch from 373b81b to 1931b14 Compare April 11, 2016 08:53
@JhonnyL JhonnyL changed the title [DependencyInjection] Add coverage for all invalid Expression in exportParameters [DependencyInjection] Add coverage for invalid Expression in exportParameters Apr 11, 2016
@fabpot
Copy link
Member

fabpot commented Apr 11, 2016

Thank you @JhonnyL.

@fabpot fabpot merged commit 1931b14 into symfony:2.7 Apr 11, 2016
fabpot added a commit that referenced this pull request Apr 11, 2016
@fabpot
minor #18499 [DependencyInjection] Add coverage for invalid Expressio…
8bf4969 
…n in exportParameters (JhonnyL)

This PR was merged into the 2.7 branch.

Discussion
----------

[DependencyInjection] Add coverage for invalid Expression in exportParameters

| Q             | A
| ------------- | ---
| Branch?       | 2.7
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| License       | MIT

Commits
-------

1931b14 [DependencyInjection] Add coverage for invalid Expression in exportParameters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
DependencyInjection Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@JhonnyL @fabpot @javiereguiluz @carsonbot