Skip to content
This repository has been archived by the owner on Dec 11, 2020. It is now read-only.

Add hautelook/AliceBundle #634

Merged
merged 1 commit into from
Jul 13, 2015
Merged

Add hautelook/AliceBundle #634

merged 1 commit into from
Jul 13, 2015

Conversation

theofidry
Copy link
Contributor

No description provided.

@fzaninotto
Copy link
Owner

What's the difference with the other bundle?

@theofidry
Copy link
Contributor Author

Historically hautelook/AliceBundle was developed first but h4cc was a little bit frustrated as there was some functionalities lacking and the bundle not very active. But now both are no longer very active which is a bit annoying.

As nelmio/alice released a new version 2.0, I'm currently discussing with hautelook/AliceBundle to upgrade the version cf. my fork and discussing with its maintainer to keep this bundle living.

But I'm not sure it's really the point: this part is a list of third-party libraries relying on this library. Shouldn't we let to the choice of which library should be used to the user?

@fzaninotto
Copy link
Owner

You're right, but repeating the same description doesn't help. I'd prefer if you listed the two names, and give a single description, something like:

Foo, Bar: Description

@theofidry
Copy link
Contributor Author

Changed. I regrouped them under a Symfony2. In the end even BazingaFakerBundle does pretty much the same as the two other, although does not rely on fixtures loader.

fzaninotto added a commit that referenced this pull request Jul 13, 2015
@fzaninotto fzaninotto merged commit 587a38f into fzaninotto:master Jul 13, 2015
@fzaninotto
Copy link
Owner

Thanks!

@theofidry
Copy link
Contributor Author

You're welcome :)

@theofidry theofidry deleted the patch-2 branch July 13, 2015 13:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants