Skip to content

Fix SUPPORTED_LOCALES array syntax#392

Open
imAaryash wants to merge 1 commit intosiddharthvaddem:mainfrom
imAaryash:patch-1
Open

Fix SUPPORTED_LOCALES array syntax#392
imAaryash wants to merge 1 commit intosiddharthvaddem:mainfrom
imAaryash:patch-1

Conversation

@imAaryash
Copy link
Copy Markdown

@imAaryash imAaryash commented Apr 8, 2026

Description

removed extra comma which was breaking the syntax

Type of Change

  • Refactor / Code Cleanup

Summary by CodeRabbit

  • Bug Fixes
    • Corrected a configuration issue affecting the language support system.

Copilot AI review requested due to automatic review settings April 8, 2026 13:50
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 710641f8-931d-4a73-ac69-2060203e7d14

📥 Commits

Reviewing files that changed from the base of the PR and between e7d5f51 and fdfeb51.

📒 Files selected for processing (1)
  • src/i18n/config.ts

📝 Walkthrough

Walkthrough

Removed a stray empty array element from the SUPPORTED_LOCALES constant, fixing a syntax error in the locale configuration. The array now contains exactly five valid locale values instead of six with a gap.

Changes

Cohort / File(s) Summary
Locale Config
src/i18n/config.ts
Removed invalid empty array element from SUPPORTED_LOCALES constant, correcting malformed syntax from ["en","zh-CN","es", , "fr","tr"] to ["en","zh-CN","es","fr","tr"].

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🎯 a ghost in the array,
hiding between the commas—
now it's gone, gone, gone.
five locales speak true,
no phantoms allowed. ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: fixing invalid array syntax in SUPPORTED_LOCALES by removing an errant comma.
Description check ✅ Passed The description covers the core issue (removed extra comma breaking syntax) and marks the change type, but lacks detail on motivation, testing, and other template sections.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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
Copy Markdown

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

Fixes a syntax error in the i18n configuration by removing an extraneous comma from the SUPPORTED_LOCALES tuple, restoring valid TypeScript parsing and ensuring locale-related UI and logic can iterate the list correctly.

Changes:

  • Removed an invalid empty element from the SUPPORTED_LOCALES array in src/i18n/config.ts.

@imAaryash
Copy link
Copy Markdown
Author

Hii @siddharthvaddem this broken syntax was failing the production builds

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