Skip to content

[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