Skip to content

fix: format taxa list files to pass Prettier check#1161

Merged
mihow merged 1 commit intomainfrom
fix/prettier-taxa-list
Feb 27, 2026
Merged

fix: format taxa list files to pass Prettier check#1161
mihow merged 1 commit intomainfrom
fix/prettier-taxa-list

Conversation

@mihow
Copy link
Collaborator

@mihow mihow commented Feb 27, 2026

Summary

  • Fixes failing lint job on main (the Check Format step)
  • Runs Prettier on two files introduced in Configurable taxa lists #1094 that had long ternary expressions exceeding the print width

The formatting wasn't caught because the final commit in #1094 was merged before CI completed on that commit.

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings February 27, 2026 11:38
@netlify
Copy link

netlify bot commented Feb 27, 2026

Deploy Preview for antenna-ssec ready!

Name Link
🔨 Latest commit a0b3141
🔍 Latest deploy log https://app.netlify.com/projects/antenna-ssec/deploys/69a1822bfbf79c0008101ce0
😎 Deploy Preview https://deploy-preview-1161--antenna-ssec.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Feb 27, 2026

Deploy Preview for antenna-preview ready!

Name Link
🔨 Latest commit a0b3141
🔍 Latest deploy log https://app.netlify.com/projects/antenna-preview/deploys/69a1822b852dc60008f552c7
😎 Deploy Preview https://deploy-preview-1161--antenna-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 60 (🔴 down 6 from production)
Accessibility: 80 (no change from production)
Best Practices: 92 (🔴 down 8 from production)
SEO: 92 (no change from production)
PWA: 80 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 27, 2026

Warning

Rate limit exceeded

@mihow has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 13 minutes and 26 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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.

📥 Commits

Reviewing files that changed from the base of the PR and between 0487cf9 and a0b3141.

📒 Files selected for processing (2)
  • ui/src/pages/taxa-list-details/add-taxa-list-taxon/add-taxa-list-taxon.tsx
  • ui/src/pages/taxa-list-details/taxa-list-details.tsx
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/prettier-taxa-list

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 setDetailBreadcrumb in the taxa list details page.
  • Reformats a ternary used for the TaxonSelect triggerLabel prop 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.

@mihow mihow merged commit 12a3c70 into main Feb 27, 2026
11 checks passed
@mihow mihow deleted the fix/prettier-taxa-list branch February 27, 2026 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants