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

Skip to content

SensioGeneratorBundle vs SymfonyMakerBundle#9830

Merged
pamil merged 2 commits into
Sylius:1.3from
davidroberto:patch-3
Oct 18, 2018
Merged

SensioGeneratorBundle vs SymfonyMakerBundle#9830
pamil merged 2 commits into
Sylius:1.3from
davidroberto:patch-3

Conversation

@davidroberto
Copy link
Copy Markdown
Contributor

@davidroberto davidroberto commented Oct 15, 2018

Sensio Generator Bundle does not support Symfony 4, and we should use the Maker Bundle as it's stated here https://github.com/sensiolabs/SensioGeneratorBundle

Q A
Branch? 1.2, 1.3 or master
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets fixes #X, partially #Y, mentioned in #Z
License MIT

Sensio Generator Bundle does not support Symfony 4, and we should use the Maker Bundle as it's stated here https://github.com/sensiolabs/SensioGeneratorBundle
@davidroberto davidroberto requested a review from a team as a code owner October 15, 2018 13:01
@Zales0123 Zales0123 added the Documentation Documentation related issues and PRs - requests, fixes, proposals. label Oct 15, 2018
@pamil
Copy link
Copy Markdown
Contributor

pamil commented Oct 16, 2018

Thanks for the PR! I feel like it's a good move to switch to recommending the MakerBundle in Sylius 1.3. Could you also update the rest of that file with instructions for MakerBundle? It's okay if you couldn't, then I'd merge this one and do it myself probably :)

@davidroberto
Copy link
Copy Markdown
Contributor Author

Yeah no problem, i'll add some infos about the MakerBundle :)

add step for creating an entity with the maker bundle
@davidroberto
Copy link
Copy Markdown
Contributor Author

Done!

@pamil pamil merged commit caef7b7 into Sylius:1.3 Oct 18, 2018
@pamil
Copy link
Copy Markdown
Contributor

pamil commented Oct 18, 2018

Thank you, David! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Documentation related issues and PRs - requests, fixes, proposals.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants