Skip to content
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

Feature/op cache check #42

Merged
merged 5 commits into from
Nov 6, 2023
Merged

Feature/op cache check #42

merged 5 commits into from
Nov 6, 2023

Conversation

david-d-h
Copy link
Contributor

Deze PR voegt de OpCacheCheck toe. Het checkt of OpCache enabled is door middel van de opcache_get_status functie.

@david-d-h david-d-h added the enhancement New feature or request label Oct 16, 2023
@david-d-h david-d-h self-assigned this Oct 16, 2023
@david-d-h david-d-h requested a review from markvaneijk October 16, 2023 09:16
@markvaneijk
Copy link
Contributor

@dulkoss top! Voor de CLI is er wel een aparte status attribuut in die array die je terugkrijgt, hier moet je ook nog op checken.

@markvaneijk
Copy link
Contributor

Zou daarnaast graag een method erbij willen hebben, waarmee je aangeeft ook te willen checken of de JIT compiler aan staat. Standaard hoeft de check hier niet op te controleren.

@markvaneijk markvaneijk merged commit 0112d87 into main Nov 6, 2023
@markvaneijk markvaneijk deleted the feature/op-cache-check branch November 6, 2023 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants