Format AIML representative model list#967
Format AIML representative model list#967PeterDaveHello wants to merge 1 commit intoChatGPTBox-dev:masterfrom
Conversation
Collapse the AIML representative model list to Prettier's single-line form so later commits do not pick up unrelated formatting churn. The drift came from 0eb64f7, which refreshed the AIML model IDs but left the older multiline layout from ea210a9. Reference: - 0eb64f7 Refresh AIML default model presets - ea210a9 Expand OpenAI model regression coverage
Review Summary by QodoFormat AIML representative model list to single-line WalkthroughsDescription• Collapse AIML representative model list to single-line format • Align with Prettier's formatting standard • Prevent unrelated formatting churn in future commits Diagramflowchart LR
multiline["Multiline AIML model list"] -- "collapse to single-line" --> singleline["Single-line AIML model list"]
File Changes1. tests/unit/config/config-predicates.test.mjs
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughA test fixture in the config predicates test file was reformatted from a multi-line array declaration to a single-line format. No logic, assertions, or functionality was altered. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~1 minute Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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. Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.Comment |
There was a problem hiding this comment.
Pull request overview
Normalizes formatting of the AIML representative model list in the config predicate unit tests to match the repo’s Prettier output, reducing future formatting-only churn.
Changes:
- Reformat
representativeAimlApiModelNamesfrom a multiline array to a single-line array (fits within configuredprintWidth: 100).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Collapse the AIML representative model list to Prettier's single-line form so later commits do not pick up unrelated formatting churn.
The drift came from 0eb64f7, which refreshed the AIML model IDs but left the older multiline layout from ea210a9.
Reference:
Summary by CodeRabbit
Release Notes