fix: format taxa list files to pass Prettier check#1161
Conversation
Co-Authored-By: Claude <noreply@anthropic.com>
✅ Deploy Preview for antenna-ssec ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for antenna-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Pull request overview
This PR fixes a failing UI formatting/lint check by applying Prettier-compliant formatting to taxa list UI files introduced in #1094, specifically addressing long inline ternary expressions that exceeded the configured print width.
Changes:
- Reformats a ternary passed to
setDetailBreadcrumbin the taxa list details page. - Reformats a ternary used for the
TaxonSelecttriggerLabelprop in the add-taxon UI.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| ui/src/pages/taxa-list-details/taxa-list-details.tsx | Splits a long ternary into multi-line Prettier formatting for breadcrumb title. |
| ui/src/pages/taxa-list-details/add-taxa-list-taxon/add-taxa-list-taxon.tsx | Wraps/splits the triggerLabel ternary to satisfy Prettier print width. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Summary
lintjob on main (theCheck Formatstep)The formatting wasn't caught because the final commit in #1094 was merged before CI completed on that commit.