-
Notifications
You must be signed in to change notification settings - Fork 137
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
Get rid of phive #881
Get rid of phive #881
Conversation
The failed checks are not using the changed workflow in this PR, that's why they are failing. |
@@ -19,8 +19,8 @@ | |||
"require": { | |||
"php": "^7.4 || ^8.0", | |||
"ext-json": "*", | |||
"composer/xdebug-handler": "^3.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.
To make things work, I had to change some packages' versions. I have tested that they are still working as expected.
@@ -15,6 +15,9 @@ var_dump($env->get('BAR')); | |||
var_dump($env->get('TEST')); | |||
|
|||
--EXPECT-- | |||
Standard input code:7: |
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.
I guess this is what you meant when talking about the extra PHPUnit output. I tried to figure out what causes that, but in the end I said to myself that it doesn't really matter anyway.
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.
Looks great. Thanks for taking care of this and updating the docs as well. I will merge it soon-ish.
Does this mean that Phive installation is no longer supported? Or is this just about this package using Phive? |
It's about this package using phive. You should still be able to install deptrac via phive yourself. |
Closes: #785