Skip to content

init:bundle => bundle:init #125

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

Closed
wants to merge 1 commit into from
Closed

Conversation

Seldaek
Copy link
Member

@Seldaek Seldaek commented Feb 27, 2011

This should be merged if/when symfony/symfony#117 is merged

@kriswallsmith
Copy link
Contributor

I would rather keep the init: namespace and and the following commands to it:

  • init:front-controller for creating front controllers in your web directory (I will send a PR for this)
  • init:container-extension for bootstrapping a DIC extension
  • init:core-bootstrap what's currently in the sandbox's 'build_bootstrap.php` script

I think it makes sense to group all code-generation commands under a common namespace to raise their visibility to the beginning user.

@Seldaek
Copy link
Member Author

Seldaek commented Feb 27, 2011

Yeah, I thought of that too, but it's semantically incorrect WRT the other commands (as said in symfony/symfony#117). And if we add other stuff to bundle, like bundle:enable for example. Then you'd have init:bundle and bundle:enable, which would suck. I agree with you for now but I think grouping by resource type is more future-proof.

@kriswallsmith
Copy link
Contributor

I didn't realize this was the docs repo :)

@lsmith77
Copy link
Contributor

lsmith77 commented Apr 5, 2011

i guess this can be closed now since symfony/symfony#117 was closed

@fabpot fabpot closed this Apr 5, 2011
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