Conversation
This commit implements issue #28 by adding a formal Laravel Facade and convenient static shorthand methods for pull request operations. Changes: - Created `Facades/PullRequests` facade extending Laravel's Facade class - Added static shorthand methods: `open()`, `closed()`, `merged()` - Renamed instance methods to avoid conflicts: `listOpen()`, `listClosed()` - Updated ServiceProvider to register facade binding - Added comprehensive test coverage for facade functionality - Added `illuminate/support` as dev dependency for facade support The facade provides an ergonomic API following Laravel conventions: ```php // Static query builders PullRequests::open('owner/repo')->get(); PullRequests::closed('owner/repo')->author('user')->get(); PullRequests::merged('owner/repo')->take(10)->get(); // Existing static methods still work PullRequests::for('owner/repo')->open()->get(); PullRequests::find('owner/repo', 123); ``` All tests pass with 100% code coverage. Resolves #28
|
Warning Rate limit exceeded@jordanpartridge has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 9 minutes and 36 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 (6)
✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
Summary
Implements #28 by adding a formal Laravel Facade and convenient static shorthand methods for pull request operations.
Changes
Core Implementation
Facades/PullRequests- Formal Laravel Facade extendingIlluminate\Support\Facades\Facadeopen(),closed(),merged()for convenient accesslistOpen(),listClosed()to avoid naming conflicts with static methodsilluminate/supportas dev dependency for facade supportAPI Examples
Benefits
Test Coverage
Quality Gates
Closes #28