Conversation
|
Caution Review failedFailed to post review comments 📝 WalkthroughWalkthroughThis pull request introduces a comprehensive refactor of Appwrite's routing and API infrastructure, with updates to documentation, configuration, and database schemas. Changes include migrations from Estimated code review effort🎯 4 (Complex) | ⏱️ ~75 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
Security Scan Results for PRDocker Image Scan Results
Source Code Scan Results🎉 No vulnerabilities found! |
✨ Benchmark results
⚡ Benchmark Comparison
|
🔄 PHP-Retry SummaryFlaky tests detected across commits: Commit
|
| Test | Retries | Total Time | Details |
|---|---|---|---|
LegacyConsoleClientTest::testUpdateAttributeEnum |
1 | 240.50s | Logs |
LegacyCustomServerTest::testCreatedBefore |
1 | 240.31s | Logs |
TablesDBConsoleClientTest::testAttributeResponseModels |
1 | 243.78s | Logs |
TablesDBTransactionsConsoleClientTest::testCreateDocument |
1 | 240.94s | Logs |
UsageTest::testDatabaseStatsCollectionsAPI |
1 | 10.19s | Logs |
RealtimeConsoleClientTest::testDeleteIndexCollectionsAPI |
1 | 120.13s | Logs |
And adding telemetry instead