added mb_ pollyfils and added a test suite to ensure it still works#172
Conversation
|
Warning Rate limit exceeded@gin0115 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 8 minutes and 19 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
WalkthroughThe GitHub Actions workflow for unit tests was updated by renaming the PHP versions matrix key from Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Join our Discord community for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Changes proposed in this Pull Request
This pull request introduces changes to improve compatibility with PHP environments that may lack the
mbstringextension. It updates the CI workflow to handle cases wherembstringis unavailable and adds polyfills formb_strimwidthandmb_strleninfunctions.php. These changes ensure functionality even in environments withoutmbstring.CI Workflow Updates:
.github/workflows/unit-tests.yml: Modified the matrix strategy to include a special case for PHP 8.3 without thembstringextension. Updated the setup step to dynamically configure PHP and extensions based on the matrix values. [1] [2]Polyfills for Missing Functions:
functions.php: Added polyfills formb_strimwidthandmb_strlento provide fallback implementations when thembstringextension is not available. These polyfills ensure compatibility and graceful degradation.Testing instructions
Mentions #
Summary by CodeRabbit
New Features
mb_strimwidthandmb_strlenfunctions to improve compatibility in environments without native multibyte string support.Chores
mbstringPHP extension based on PHP version and display the current PHP version during testing.