Skip to content

Fix doubled words in locale-check route comment#3681

Open
J8118 wants to merge 1 commit intoShopify:mainfrom
J8118:fix/doubled-words-locale-check
Open

Fix doubled words in locale-check route comment#3681
J8118 wants to merge 1 commit intoShopify:mainfrom
J8118:fix/doubled-words-locale-check

Conversation

@J8118
Copy link
Copy Markdown
Contributor

@J8118 J8118 commented Apr 10, 2026

Title

Fix doubled words in locale-check route comment

Description

Summary

Two doubled-word typos in the comment block of locale-check.ts:

  • Line 10: "yet we still are still at the default locale""yet we are still at the default locale"
  • Line 11: "then the the locale param must be invalid""then the locale param must be invalid"

Files changed:

  • packages/cli/assets/routes/locale-check.ts — comment-only fix

Note: This same text also appears in packages/cli/CHANGELOG.md:792-793 and templates/skeleton/CHANGELOG.md:2486-2487 as part of historical changelog entries. Those are left as-is since changelogs are historical records.

Fix "we still are still" → "we are still" and
"the the locale" → "the locale" in comment block.
@J8118 J8118 requested a review from a team as a code owner April 10, 2026 07:26
@J8118
Copy link
Copy Markdown
Contributor Author

J8118 commented Apr 10, 2026

I have signed the CLA!

Copy link
Copy Markdown
Contributor

@fredericoo fredericoo left a comment

Choose a reason for hiding this comment

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

LGTM - doubled words in a comment, clear fix.

@fredericoo
Copy link
Copy Markdown
Contributor

cheers for the fix! this is currently failing CI because the branch is based on an older main - could you rebase onto the latest main so we can get this merged? ta!

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.

3 participants