fix: user sending the wrong command is not an error#8270
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR refines error handling for non-interactive CLI command suggestions. It adds a new error message pattern ( Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsLinked repositories: Couldn't analyze Comment |
🎉 Thanks for submitting a pull request! 🎉
Summary
Right now we throw an error if the user is using a wrong command on the CLI. Not only does this give a lot of noise in our monitoring, it is also not necessary.
This changes the logic to only log and exit, and also to filter out these kind of messages from the error logging (for old versions still doing this)
For us to review and ship your PR efficiently, please perform the following steps:
passes our tests.
A picture of a cute animal (not mandatory, but encouraged)