chore(ui-avatar): Review build configuration#37608
Conversation
|
Looks like this PR is not ready to merge, because of the following issues:
Please fix the issues and try again If you have any trouble, please check the PR guidelines |
|
WalkthroughConfiguration and build setup standardization for the ui-avatar package, including ESLint configuration simplification, packaging metadata consolidation, build script definitions, and refactoring of imports to use type-only declarations for type-related imports. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro Disabled knowledge base sources:
⛔ Files ignored due to path filters (1)
📒 Files selected for processing (5)
💤 Files with no reviewable changes (1)
🧰 Additional context used📓 Path-based instructions (1)**/*.{ts,tsx,js}📄 CodeRabbit inference engine (.cursor/rules/playwright.mdc)
Files:
🧠 Learnings (2)📓 Common learnings📚 Learning: 2025-11-17T15:07:13.273ZApplied to files:
🔇 Additional comments (4)
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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #37608 +/- ##
===========================================
+ Coverage 68.83% 68.85% +0.01%
===========================================
Files 3361 3361
Lines 114340 114340
Branches 20619 20619
===========================================
+ Hits 78706 78726 +20
+ Misses 33536 33520 -16
+ Partials 2098 2094 -4
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Proposed changes (including videos or screenshots)
It normalizes the ESLint configuration for
@rocket.chat/ui-avatar.Issue(s)
ARCH-1893
Steps to test or reproduce
Further comments
Summary by CodeRabbit
Release Notes
Chores
Refactor
✏️ Tip: You can customize this high-level summary in your review settings.