-
-
Notifications
You must be signed in to change notification settings - Fork 154
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
Fix: Set cacheResult attribute to false in phpunit.xml #585
Conversation
@@ -107,6 +107,7 @@ public function build(string $version): string | |||
$this->xmlConfigurationHelper->replaceWithAbsolutePaths($xPath); | |||
$this->xmlConfigurationHelper->setStopOnFailure($xPath); | |||
$this->xmlConfigurationHelper->deactivateColours($xPath); | |||
$this->xmlConfigurationHelper->deactivateResultCaching($xPath); | |||
$this->xmlConfigurationHelper->removeCacheResultFile($dom, $xPath); |
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.
The question is whether we still need to remove the cacheResultFile
attribute, then.
What do you think?
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.
Seems like it's not needed anymore. I would remove this code
1bae9b6
to
6d85f5b
Compare
All new files must pass this check, but for the old ones, I think we can ignore it (it's unrelated to your PR). Of course, increasing the MSI is a good thing though. Probably we need target |
6d85f5b
to
18e5f08
Compare
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.
Cool, thank you @localheinz
Thank you, @borNfreee! |
Released 0.11.4 |
This PR:
cacheResult
attribute tofalse
inphpunit.xml
to disable result caching entirelycacheResultFile
attribute fromphpunit.xml
Follows #582.