-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Migrate opcache check to new SetupCheck API #41547
Conversation
47bf97f
to
09c67c5
Compare
09c67c5
to
95ea46c
Compare
7ee93fb
to
a9c6381
Compare
a9c6381
to
114c0eb
Compare
For the |
Signed-off-by: Côme Chilliet <[email protected]>
Signed-off-by: Côme Chilliet <[email protected]>
Signed-off-by: Côme Chilliet <[email protected]>
Signed-off-by: Côme Chilliet <[email protected]>
72b1474
to
e832c14
Compare
Signed-off-by: Côme Chilliet <[email protected]>
e832c14
to
94d0399
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.
Anything changed beside moving code and adapting tests?
Failure not related, see #42633 |
Yes, level is error or warninig depending on which recommendation is triggered, saw that with Benjamin. |
Summary
Migrate opcache check to new SetupCheck API
TODO
<code>
tags.Checklist