Skip to content
This repository was archived by the owner on Sep 16, 2021. It is now read-only.

we instruct users to move the composer.phar to composer #590

Merged
merged 2 commits into from
Oct 25, 2014
Merged

Conversation

lsmith77
Copy link
Member

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets -

@lsmith77
Copy link
Member Author

related to this .. should be also move to suggesting composer require?
https://github.com/thephpleague/skeleton/blob/master/README.md#install

@wouterj
Copy link
Member

wouterj commented Oct 22, 2014

We should update all bundle docs to comply with http://symfony.com/doc/current/cookbook/bundles/best_practices.html#installation-instructions But let's handle that in another PR.

@lsmith77
Copy link
Member Author

ok, then lets merge?

@lsmith77
Copy link
Member Author

hmm tough this should go into master as well I guess? though I guess we should now also merge dev into master ..?

@dbu
Copy link
Member

dbu commented Oct 22, 2014

@wouterj was discussing on this in #550 - it seems the same topic to me.

lets wait with merging dev to master until we tagged symfony-cmf/symfony-cmf (waiting for routing auto) and the sandbox and SE.

@lsmith77
Copy link
Member Author

routing auto is done now too. so I guess we could already tag the SE .. but for the sandbox we also need the sonata bundle ..

wouterj added a commit that referenced this pull request Oct 25, 2014
we instruct users to move the composer.phar to composer
@wouterj wouterj merged commit 90e3987 into dev Oct 25, 2014
@wouterj wouterj deleted the composer_phar branch October 25, 2014 15:14
@wouterj wouterj mentioned this pull request Oct 25, 2014
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.

3 participants