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

Autoloading via composer #18

Closed
hrach opened this issue Dec 5, 2016 · 5 comments
Closed

Autoloading via composer #18

hrach opened this issue Dec 5, 2016 · 5 comments

Comments

@hrach
Copy link
Contributor

hrach commented Dec 5, 2016

PHPStan IMO should not depend on composer autoloader of the project. It is not a direct (dev) dependency and no code depends on it. That's the main difference to other dev tools such as PHPUnit, the code is directly dependent on PHPUnit.

Would be great to use it as "tool" in CI:

composer create-project phpstan/phpstan --no-dev
./phpstan/phpstan analyse ./myproject
@ondrejmirtes
Copy link
Member

Yep, I'd like to to that really soon :)

@kix
Copy link

kix commented Dec 6, 2016

By the way, currently PHPStan is also unusable as an external tool this way, composer global require phpstan/phpstan wouldn't work.

@ondrejmirtes
Copy link
Member

Yep, it isn't, it will be an objective of a future release.

@shouze
Copy link

shouze commented Dec 29, 2016

#50 is a proposition to fix that. We want to use phpstan as an external tool and we can do that with this PR.

@ondrejmirtes
Copy link
Member

This is now possible thanks to 3c71eda and 57f2ebe.

@lock lock bot locked as resolved and limited conversation to collaborators Dec 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants