test: add type compiler tests for ThemeQuickPresets#8123
Conversation
|
This PR introduces important tests for the ThemeQuickPresets, which is crucial for maintaining the integrity of the theme design. Similar past decisions have focused on adding tests for timezone normalization and responsive layouts, emphasizing the importance of thorough testing in our codebase. Ensuring that our components are well-tested helps us maintain high quality and reliability. Thank you for your contribution, and I look forward to your updates! |
📦 Next.js Bundle Size Report (Gzipped Sizes)✨ No significant bundle size changes detected. 📊 Summary of Totals
|
Aamod007
left a comment
There was a problem hiding this comment.
Great job adding type compiler validations for the \ThemeQuickPresets\ component. Ensuring the structure of \Props['theme']\ and the \onThemeChange\ callback signature are correct provides excellent static typing guarantees. The tests are well-structured and concise. Keep it up!
|
🎉 Congratulations @deepsikha-dash! Your PR has been successfully merged. 🚀 Thank you for contributing to CommitPulse. Your work helps us build a better tool for the community.
Keep building! 💻✨ |
Description
Fixes #6887
Pillar
Visual Preview
N/A
Checklist before requesting a review:
CONTRIBUTING.mdfile.localhost:3000/api/streak?user=YOUR_USERNAME).npm run formatandnpm run lintlocally and resolved all errors (CI will fail otherwise).feat(themes): ...,fix(calculate): ...).README.mdif I added a new theme or URL parameter.