-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[Integration Tests] Clear opcode cache after config change #4302
Conversation
@LukasReschke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @nickvergessen, @schiessle and @ChristophWurst to be potential reviewers. |
soooo… what did you debug/test here? I'm just curious … |
😃 |
Also failing on #4304 … mhm… |
If all failes, use the testing apps api |
Yeah. I suspect the OPCode cache to be to blame… Let me add a small PHP file to the testing app that cleans the OPCode. For testing purposes only of course :-) |
5799116
to
925e05d
Compare
Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
925e05d
to
4cf6148
Compare
Codecov Report
@@ Coverage Diff @@
## master #4302 +/- ##
=========================================
Coverage ? 54.14%
Complexity ? 21395
=========================================
Files ? 1327
Lines ? 81789
Branches ? 1304
=========================================
Hits ? 44287
Misses ? 37502
Partials ? 0
Continue to review full report at Codecov.
|
Ok. This seems to work now 🚀 |
Let's see…