Conversation
|
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 2 Skipped Deployments
|
|
Important Review skippedMore than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review. 118 files out of 271 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits. You can disable this status message by setting the ✨ Finishing Touches
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page 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 (
|
chronark
left a comment
There was a problem hiding this comment.
ok first pass is done, let's go through all open discussions and close or turn them into issues via coderabbit
|
I think it went trough all the comments and either fixed them or let rabbit create issues. |
What does this PR do?
For #3419
Fixes #3493
Fixes #3465
adds the key service which we can use for verifiy api keys
makes s3 (vault) optional
Improves caching for ratelimit overrides
adds some seed helper utilities
adjusted some indexes to de-duplicate them since we should always search with a identifier id e.g workspace / key / identity
Bug fix (non-breaking change which fixes an issue)
Chore (refactoring code, technical debt, workflow improvements)
Enhancement (small improvements)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update
How should this be tested?
Unit Tests
Checklist
Required
pnpm buildpnpm fmtconsole.logsgit pull origin mainAppreciated