- 
          
- 
                Notifications
    You must be signed in to change notification settings 
- Fork 126
chore: update pnpm to latest 9.x #2212
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| 📝 WalkthroughWalkthroughThe prerequisites section in the  Changes
 Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Note 🔌 MCP (Model Context Protocol) integration is now available in Early Access!Pro users can now connect to remote MCP servers under the Integrations page to get reviews and chat conversations that understand additional development context. ✨ Finishing Touches🧪 Generate unit tests
 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. CodeRabbit Commands (Invoked using PR comments)
 Other keywords and placeholders
 CodeRabbit Configuration File ( | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
🧹 Nitpick comments (2)
CONTRIBUTING.md (2)
3-3: Typo: “think” → “thank”.Small wording fix.
-I want to think you first for considering contributing to ZenStack 🙏🏻. It's people like you who make ZenStack a better toolkit that benefits more and more developers! This document will help you get started, and please join our [Discord](https://discord.gg/Ykhr738dUe) if you have any questions. +I want to thank you first for considering contributing to ZenStack 🙏🏻. It's people like you who make ZenStack a better toolkit that benefits more and more developers! This document will help you get started, and please join our [Discord](https://discord.gg/Ykhr738dUe) if you have any questions.
10-10: Grammar/clarity: fix phrasing for test DB setup/teardown and connection details.Improves readability and accuracy.
-If you want to run the tests, you should be aware that some of the integration tests run against postgres. These tests will attempt to set up and subsequently their own database, so you'll need to provide a connection details for a postgres user with at least those permissions. To provide connection details, you can configure the following environment variables or provide them when executing `pnpm test` commands. +If you want to run the tests, note that some integration tests run against Postgres. These tests will attempt to set up and subsequently tear down their own database, so you'll need to provide connection details for a Postgres user with permission to create and drop databases. You can provide the connection details by setting the following environment variables or by passing them when executing `pnpm test` commands.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (4)
- .github/workflows/build-test.ymlis excluded by- !**/*.yml
- .github/workflows/integration-test.ymlis excluded by- !**/*.yml
- .github/workflows/regression-test.ymlis excluded by- !**/*.yml
- package.jsonis excluded by- !**/*.json
📒 Files selected for processing (1)
- CONTRIBUTING.md(1 hunks)
No description provided.