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

Skip to content

[symfony/HttpKernel] Throws an error when the generated class name is invalid.#30906

Merged
fabpot merged 1 commit intosymfony:masterfrom
drupol:30845-invalid-classname-generation
Apr 6, 2019
Merged

[symfony/HttpKernel] Throws an error when the generated class name is invalid.#30906
fabpot merged 1 commit intosymfony:masterfrom
drupol:30845-invalid-classname-generation

Conversation

@drupol
Copy link
Contributor

@drupol drupol commented Apr 6, 2019

Q A
Branch? master
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #30845
License MIT

@nicolas-grekas nicolas-grekas added this to the next milestone Apr 6, 2019
@drupol drupol changed the title Issue #30845: Throws an error when the generated class name is invalid. [symfony/HttpKernel] Throws an error when the generated class name is invalid. Apr 6, 2019
@fabpot
Copy link
Member

fabpot commented Apr 6, 2019

Thank you @drupol.

@fabpot fabpot merged commit c976866 into symfony:master Apr 6, 2019
fabpot added a commit that referenced this pull request Apr 6, 2019
…d class name is invalid. (drupol)

This PR was squashed before being merged into the 4.3-dev branch (closes #30906).

Discussion
----------

[symfony/HttpKernel] Throws an error when the generated class name is invalid.

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #30845
| License       | MIT

Commits
-------

c976866 [symfony/HttpKernel] Throws an error when the generated class name is invalid.
@drupol drupol deleted the 30845-invalid-classname-generation branch April 6, 2019 15:53
alquerci added a commit to alquerci/symfony that referenced this pull request Apr 10, 2019
…enerated class name is invalid. (drupol)
@nicolas-grekas nicolas-grekas modified the milestones: next, 4.3 Apr 30, 2019
@fabpot fabpot mentioned this pull request May 9, 2019
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