Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[Workflow] fixed InvalidDefinitionException message for StateMachineValidator #23865

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
Aug 11, 2017
Merged

[Workflow] fixed InvalidDefinitionException message for StateMachineValidator #23865

merged 1 commit into from
Aug 11, 2017

Conversation

fmata
Copy link
Contributor

@fmata fmata commented Aug 11, 2017

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

Little mistake in the exception message which can be lead to difficult debugging if we don't pay attention in the workflow definition.

@lyrixx
Copy link
Member

lyrixx commented Aug 11, 2017

👍

@lyrixx
Copy link
Member

lyrixx commented Aug 11, 2017

Good catch, thanks @fmata.

@lyrixx lyrixx merged commit 9c29f9c into symfony:3.3 Aug 11, 2017
lyrixx added a commit that referenced this pull request Aug 11, 2017
…ateMachineValidator (fmata)

This PR was merged into the 3.3 branch.

Discussion
----------

[Workflow] fixed InvalidDefinitionException message for StateMachineValidator

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

Little mistake in the exception message which can be lead to difficult debugging if we don't pay attention in the workflow definition.

Commits
-------

9c29f9c [Workflow] fixed InvalidDefinitionException message for StateMachineValidator
@fmata fmata deleted the workflow_validator branch August 11, 2017 13:28
@fabpot fabpot mentioned this pull request Aug 28, 2017
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.

3 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.