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
Ensure BC of API, compat with roave/you-are-using-it-wrong
, completeness of composer.json
#2140
Ensure BC of API, compat with roave/you-are-using-it-wrong
, completeness of composer.json
#2140
Conversation
Btw, do you already have github actions enabled for this repo? They are quite a lot faster 👍 |
|
||
git clone git@github.com:roave/you-are-using-it-wrong.git | ||
cd you-are-using-it-wrong | ||
composer install |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to run on a PHP 7.2+ circle-ci instance, so it fails:
Your requirements could not be resolved to an installable set of packages.
Problem 1
- This package requires php ^7.3.0 but your PHP version (7.2.22) does not satisfy that requirement.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feel free to bump the version it runs on!
.travis.yml
Outdated
php: 7.3 | ||
env: DEPS="high" | ||
script: | ||
- composer require --dev roave/backward-compatibility-check:^4.0.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't work due to conflicting deps: will try making a new dir and installing it there
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.
Problem 1
- Installation request for roave/backward-compatibility-check ^4.0.0 -> satisfiable by roave/backward-compatibility-check[4.0.0].
- Can only install one of: phpdocumentor/type-resolver[1.0.1, 0.4.0].
- Can only install one of: phpdocumentor/type-resolver[0.4.0, 1.0.1].
- Can only install one of: phpdocumentor/type-resolver[0.4.0, 1.0.1].
- Conclusion: install phpdocumentor/type-resolver 0.4.0
- Installation request for phpdocumentor/type-resolver (locked at 1.0.1) -> satisfiable by phpdocumentor/type-resolver[1.0.1].
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, this should just be on circleci I think
…h no other dependencies
`ocramius/proxy-manager` is an extremely heavy `vimeo/psalm` consumer, and relies on a lot of the templated types system to generate real types for proxies produced by runtime evaluation.
29e7281
to
d31cf24
Compare
Thanks! |
@muglug is it OK even though it is red? |
If it's red now I'll figure out how to make it green in time for release! Also Psalm now picks up two errors:
|
This is possibly a new inspection, right? |
No – it's one that should have run before, but wasn't, due to a bug |
I'm fine with suppressing or working around that one, since the set of known classes is known to maintainers: will do so in the downstream project. It does indeed make sense to prevent |
No description provided.