Skip to content

Conversation

@ahilke
Copy link

@ahilke ahilke commented Jun 26, 2019

winzou/state-machine-bundle 0.4.1 fixes an incompatibility with Symfony 4. Allowing users to upgrade the state-machine-bundle make using Symfony 4 easier.

ahilke added 2 commits June 26, 2019 18:48
winzou/state-machine-bundle 0.4.1 fixes an incompatibility with Symfony 4. Allowing users to upgrade the state-machine-bundle make using Symfony 4 easier.
Replace sm.factory with SM\Factory\FactoryInterface.
None of the three services in the compiler pass are retrieved directly from the container, thus they do not need to be public.
@ahilke ahilke changed the title Allow the use of WinzouStateMachineBundle 0.4.* Upgrade to WinzouStateMachineBundle ^0.4.1 Jun 26, 2019
@mamazu
Copy link
Contributor

mamazu commented Nov 28, 2019

👍 Although this might be a BC break.

@pamil
Copy link
Contributor

pamil commented Jan 2, 2020

We'd need to add a compiler pass which provides BC with 0.3 behaviour (sm.factory service etc.)

@mamazu
Copy link
Contributor

mamazu commented Jan 2, 2020

Or probably an alias will just do.

@pamil
Copy link
Contributor

pamil commented Jan 2, 2020

An alias in a compiler pass, since we'd modify services coming from outside resource bundle.

@pamil pamil closed this in #133 Jan 2, 2020
pamil added a commit that referenced this pull request Jan 2, 2020
…0 & remove support for <4.4 (pamil)

This PR was merged into the 1.6-dev branch.

Discussion
----------

Closes #134, closes #99.

We need to upgrade DoctrineBundle and FOSRestBundle (which has no support for Symfony 5 yet).

A bit of dependencies maintenance done here + a reliable build setup.

Commits
-------

474979c Fix testing multiple Symfony versions & remove support for 4.3
a43600e Allow for Winzou/StateMachineBundle ^0.4
f5f6fe9 Fix specs to be compatible with Twig 3
222c2bd Drop support for Symfony ^3.4, add support for Symfony ^5.0
ab2379f Do not require the component package in the bundle (it is replaced by it)
936c891 Remove unneccessary dependency on sylius/locale
1ab2b10 Increase memory limit for Composer
64a13d6 Allow failures for Symfony 5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants