Skip to content
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

Purpose of this tool / Why not tell people they can use regular require/require-dev instead? #36

Closed
naderman opened this issue Feb 13, 2019 · 2 comments

Comments

@naderman
Copy link

Can you help me understand what the purpose of this tool is? As far as I can see you really just end up installing per-project dependencies into a non-standard directory. If one doesn't want to share these tools across projects, why wouldn't one just use a regular require/require-dev in the project itself and avoid all of this?

May it make sense to modify the README to recommend people use require/require-dev instead of this tool if they have project specific dependencies?

@greg-1-anderson
Copy link
Member

See use of native composer features for existing documented workaround. A recommendation to use per-project dependencies could also be placed here. PRs welcome.

This project exists because folks insist on using composer global require ... to install things that are not global Composer installers. See fixing the composer global command. If you never use composer global, then you do not need cgr.

@naderman
Copy link
Author

naderman commented Feb 14, 2019

@greg-1-anderson Thanks, I did read that section which just didn't mention non-global installation so far. I'm basically trying to understand why tools like this are being built rather than trying to encourage people to use regular composer require without the global part. Depending on what the rationale is we may or may not want to incorporate some changes into Composer v2, but I found a link to the issue where this should probably be discussed in your blog post (which also doesn't mention the solution of just not using "global" require at all): composer/composer#5390 (comment) - so I'll read up and comment there instead, sorry for creating an issue here for this!

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

No branches or pull requests

2 participants