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

Cookbook entry for sandbox #50

Merged
merged 5 commits into from
Dec 1, 2012
Merged

Conversation

tiagojsag
Copy link
Contributor

Cookbook entry for sandbox

@@ -1,2 +1,3 @@
_build/
*.pyc
.project
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should go into your global git config, its ide specific, not project specific ;-)

@dbu
Copy link
Member

dbu commented Nov 28, 2012

thanks for this pull request.

after all my commenting, i start to wonder if you meant to do a cookbook
entry for the cmf standard edition and not for the sandbox. those are
two different things.

standard edition is meant as starting point for a full cmf site. sandbox
is meant to demo the features and play around.

cheers,david

@tiagojsag
Copy link
Contributor Author

No, it's for the sandbox.
Thanks for your feedback, I'll take a look at it when I have a minute

@tiagojsag
Copy link
Contributor Author

Yeah, I copied the wrong installation instructions :/
This is why I feel the installation procedures should be gathered at the docs page and refactored

@lsmith77
Copy link
Member

i think the text on the website is a bit confusing ..
i fixed the text there to just contain the bare minimum of instructions:
symfony-cmf/symfony-cmf-website@4876a5f

once this PR is merged we can also add a "for further instructions" link underneath there.

Preconditions
-------------

As Symfony CMF Sandbox is based on Symfony 2, you should make sure you
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Symfony2 without a space

@lsmith77
Copy link
Member

lsmith77 commented Dec 1, 2012

i guess this is now ready to be merged?

@lsmith77
Copy link
Member

lsmith77 commented Dec 1, 2012

this PR also fixes #49?
well i guess we should then also have a PR to the sandbox README to point to these docs once this is merged.

@tiagojsag
Copy link
Contributor Author

I think that http://cmf.symfony.com/get-started could also be somehow updated to clarify that there are several ways to get the project, "clean" and "sandbox". Regarding this cookbook entry, unless you have any suggestions, it's ready to be merged.

@lsmith77
Copy link
Member

lsmith77 commented Dec 1, 2012

agreed.

lsmith77 added a commit that referenced this pull request Dec 1, 2012
Cookbook entry for sandbox
@lsmith77 lsmith77 merged commit d03afc1 into symfony-cmf:master Dec 1, 2012
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