Skip to content
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

Use nelmio/alice's newer version #50

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

utkuaydin
Copy link

If it isn't breaking anything, AliceFixturesBundle could require Alice's new 2.0 version instead of 1.6.

@utkuaydin
Copy link
Author

It breaks some stuff apparently. I'll try to make it work.

@utkuaydin
Copy link
Author

I modified Factory so it uses Alice's new Loader. However a problem occurs while loading FixtureSets and that problem is fixed by nelmio/alice@5202ac9. But that fix isn't a part of 2.0.0 release so we will either wait for a new Alice release or require its master branch until there is a new release.

(phpunit tests were running fine for me, Travis gave this error about package signatures and Scrutinizer gave a phpcs: not found error)

@theofidry
Copy link

Checkout hautelook/AliceBundle#58!

@utkuaydin
Copy link
Author

@theofidry I will, thanks. I am disappointed to see my PR getting no attention for a month.

@theofidry
Copy link

Don't blame the guy, @h4cc may no longer have time to maintain this bundle. But he should add a note a deprecate its package though.

@utkuaydin
Copy link
Author

@theofidry Don't get me wrong, I am thankful for the bundle. He may no longer have the time or the desire to maintain it and that's absolutely fine. Letting me know that he can't merge would be a nice gesture (and that counts as attention, IMHO) but I can live without it. 😄

@h4cc
Copy link
Owner

h4cc commented Aug 18, 2015

I would like to blame my job and our customers ... 😭

Will try to have a look as soon as there is spare time. 🕐

@utkuaydin
Copy link
Author

😄 That's great, thanks!

@theofidry
Copy link

Haha now you can also merge back your work to hautelook/AliceBundle ;)

@h4cc
Copy link
Owner

h4cc commented Aug 18, 2015

@theofidry I will have a look then :)

Trivia: 2 years after my initial question https://github.com/hautelook/AliceBundle/issues/7

@theofidry
Copy link

Yes, although not as much as I would like.

The dependency to DoctrineFixturesBundle has been completely removed although you can use both of them together. For more information on this: DoctrineFixturesBundle support.

But I'm still using doctrine/data-fixtures and I don't think that will change for the use cases with Doctrine. I would like to check the support with Doctrine ODM and add a support for Propel too. So there's still some work left on this side.

@munkie
Copy link

munkie commented Sep 1, 2015

Hi, @h4cc
Are there any chances of merging this PR and releasing new version of alice-bundle?
It could be 1.0.x branch for example. And you could use 2.0.x-dev version of nelmio/alice bundle
And php minimum version should be set to 5.4 as in nelmio/alice.

@theofidry
Copy link

Curious to have you though on this @h4cc. If you plan to upgrade this bundle it's fine otherwise it would be better to note this project as deprecated (and on composer too). Did you have the time to check HautelookAliceBundle?

@theofidry
Copy link

Update on the issue @h4cc: HautelookAliceBundle has been out and stable for some time already now, so this library could be deprecated in favour of it.

I do think there's still a lot that could be done, but IMO they require deeper changes that must addressed at https://github.com/nelmio/alice level, in which case people can make it happen for alice v3.

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