Remove react-day-picker dependency and delete Calendar component impl…#325
Remove react-day-picker dependency and delete Calendar component impl…#325mikeallisonJS merged 1 commit intomainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
💤 Files with no reviewable changes (2)
📝 WalkthroughWalkthroughThis pull request removes the react-day-picker dependency from the UI package and eliminates the Calendar component that previously wrapped the DayPicker library. No other dependencies or components are modified. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Back to school! That's right—these changes are like erasing the chalkboard, my friend. We're saying goodbye to the Calendar component and the react-day-picker library it was using. No fancy footwork, no hidden tricks. Just a clean removal of 75 lines and one dependency. You gotta ask yourself: where are the consumptions of this Calendar component? 'Cause if it's not being used anywhere, this is about as straightforward as a two-fer commercial. Nice and simple. 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
📝 Coding Plan
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 |
…ementation
Summary by CodeRabbit