Skip to content

Conversation

@amotl
Copy link
Member

@amotl amotl commented Nov 13, 2025

About

Follow a deprecation warning issued by upstream crate-pdo package.

References

@coderabbitai
Copy link

coderabbitai bot commented Nov 13, 2025

Warning

Rate limit exceeded

@amotl has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 10 minutes and 14 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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.

📥 Commits

Reviewing files that changed from the base of the PR and between 283a0e4 and b8fe72a.

📒 Files selected for processing (6)
  • CHANGES.txt (1 hunks)
  • composer.json (1 hunks)
  • src/Crate/DBAL/Driver/PDOCrate/PDOConnection.php (2 hunks)
  • src/Crate/DBAL/Types/ArrayType.php (2 hunks)
  • src/Crate/DBAL/Types/MapType.php (2 hunks)
  • test/Crate/Test/DBAL/Functional/ConnectionTest.php (2 hunks)
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch use-new-class

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@amotl amotl force-pushed the use-new-class branch 3 times, most recently from cbb03be to ac2ce21 Compare November 13, 2025 12:29
@amotl amotl changed the base branch from main to fix-build-202511 November 13, 2025 12:30
@amotl amotl requested review from seut and surister November 13, 2025 12:31
@amotl amotl marked this pull request as ready for review November 13, 2025 12:31
@amotl amotl force-pushed the fix-build-202511 branch 5 times, most recently from 17f33c5 to c700ef5 Compare November 13, 2025 14:46
@amotl amotl force-pushed the fix-build-202511 branch 3 times, most recently from 2ed317c to 7fdd1f5 Compare November 13, 2025 15:52
Copy link
Member

@surister surister left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Base automatically changed from fix-build-202511 to main November 13, 2025 17:04
@amotl amotl force-pushed the use-new-class branch 2 times, most recently from ed3e882 to b3e4710 Compare November 13, 2025 17:15
@amotl amotl merged commit 276d739 into main Nov 13, 2025
12 checks passed
@amotl amotl deleted the use-new-class branch November 13, 2025 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants