Skip to content

Conversation

@lyrixx
Copy link
Member

@lyrixx lyrixx commented Jun 7, 2019

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

I forgot to remove this deps in #31824

@xavismeh
Copy link
Contributor

xavismeh commented Jun 7, 2019

You could also remove (Single|Multiple)StateMarkingStore classes, couldn't you?

@lyrixx
Copy link
Member Author

lyrixx commented Jun 7, 2019

You could also remove (Single|Multiple)StateMarkingStore classes, couldn't you?

They does not exist anymore. See #31824

@fabpot
Copy link
Member

fabpot commented Jun 7, 2019

Thank you @lyrixx.

@fabpot fabpot merged commit 28f2dac into symfony:master Jun 7, 2019
fabpot added a commit that referenced this pull request Jun 7, 2019
…lyrixx)

This PR was merged into the 5.0-dev branch.

Discussion
----------

[Workflow] Removed un-used dependency in composer.json

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

---

I forgot to remove this deps in #31824

Commits
-------

28f2dac [Workflow] Removed un-used dependency in composer.json
@lyrixx lyrixx deleted the workflow-deprec branch June 7, 2019 10:20
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