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

[TASK] Add Preg::matchAll to wrap preg_match_all with error handling #1328

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

JakeQZ
Copy link
Contributor

@JakeQZ JakeQZ commented Sep 20, 2024

This completes the set of wrappers for preg_ functions that might unexpectedly fail, and which are used in the codebase.

@JakeQZ JakeQZ added this to the 8.0.0 milestone Sep 20, 2024
@JakeQZ JakeQZ requested a review from oliverklee September 20, 2024 00:33
@JakeQZ JakeQZ self-assigned this Sep 20, 2024
@JakeQZ
Copy link
Contributor Author

JakeQZ commented Sep 20, 2024

Will likely have a merge conflict with #1327.

@coveralls
Copy link

coveralls commented Sep 20, 2024

Coverage Status

coverage: 97.026% (+0.03%) from 97.0%
when pulling 184acb8 on task/preg-match-all-error-handler
into 30900ea on main.

@JakeQZ JakeQZ force-pushed the task/preg-match-all-error-handler branch 2 times, most recently from ef5dc1a to 4dc2e7a Compare September 20, 2024 00:57
@oliverklee
Copy link
Contributor

This now indeed needs a rebase.

@JakeQZ JakeQZ force-pushed the task/preg-match-all-error-handler branch from 4dc2e7a to ee5a189 Compare September 20, 2024 16:37
This completes the set of wrappers for `preg_` functions that might unexpectedly
fail, and which are used in the codebase.
@JakeQZ JakeQZ force-pushed the task/preg-match-all-error-handler branch from ee5a189 to 184acb8 Compare September 20, 2024 16:43
@oliverklee oliverklee changed the title [TASK] Add Preg::matchAll to wrap preg_match_all with error handling [TASK] Add Preg::matchAll to wrap preg_match_all with error handling Sep 20, 2024
@oliverklee oliverklee merged commit 8ac1121 into main Sep 20, 2024
24 checks passed
@oliverklee oliverklee deleted the task/preg-match-all-error-handler branch September 20, 2024 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants