Skip to content
This repository was archived by the owner on Nov 27, 2020. It is now read-only.

Move Acme demo bundle to sensio DistributionBundle#643

Merged
fabpot merged 1 commit intosymfony:masterfrom
romainneutron:acme-demo-bundle
Mar 30, 2014
Merged

Move Acme demo bundle to sensio DistributionBundle#643
fabpot merged 1 commit intosymfony:masterfrom
romainneutron:acme-demo-bundle

Conversation

@romainneutron
Copy link
Copy Markdown
Contributor

@romainneutron
Copy link
Copy Markdown
Contributor Author

How to test this ?

First,

git clone git@github.com:romainneutron/symfony-standard.git

then edit composer.json and add:

    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/romainneutron/SensioDistributionBundle"
        }
    ],
    "require": {
        "sensio/distribution-bundle": "dev-acme-demo-bundle"
    }

finally

composer install

@romainneutron
Copy link
Copy Markdown
Contributor Author

Weird error, fabbot propose me to patch a file that bas been removed. Here is the proposed patch:

--- src/Acme/DemoBundle/Controller/WelcomeController.php    2014-03-20 17:31:33.845442703 +0000
+++ src/Acme/DemoBundle/Controller/WelcomeController.php    2014-03-20 17:31:39.109442703 +0000
@@ -13,6 +13,7 @@
          * or @Template annotation as demonstrated in DemoController.
          *
          */
+
         return $this->render('AcmeDemoBundle:Welcome:index.html.twig');
     }
 }

fabpot added a commit that referenced this pull request Mar 30, 2014
…inneutron)

This PR was merged into the 2.4-dev branch.

Discussion
----------

Move Acme demo bundle to sensio DistributionBundle

See https://github.com/symfony/symfony-standard/issues/639
Merge sensiolabs/SensioDistributionBundle#119 before this one

Commits
-------

eb5d496 Move Acme demo bundle to sensio DistributionBundle
@fabpot fabpot merged commit eb5d496 into symfony:master Mar 30, 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.

2 participants