Skip to content

fix: correct typos in comments, error messages, and identifiers#5704

Merged
johnnyjoygh merged 1 commit intousememos:mainfrom
memoclaw:fix/typos-in-comments-and-identifiers
Mar 8, 2026
Merged

fix: correct typos in comments, error messages, and identifiers#5704
johnnyjoygh merged 1 commit intousememos:mainfrom
memoclaw:fix/typos-in-comments-and-identifiers

Conversation

@memoclaw
Copy link
Contributor

@memoclaw memoclaw commented Mar 8, 2026

Summary

  • store/db/mysql: Fix "eroor" → "error" in DSN parse error message
  • plugin/storage/s3: Fix "presign put object" → "presign get object" in PresignGetObject error message (copy-paste bug)
  • plugin/idp/oauth2: Fix "failed to new http request" → "failed to create http request"
  • plugin/cron/parser: Fix "are parse in" → "are parsed in" in comment
  • proto/attachment_service: Fix "a attachment" → "an attachment" in 3 RPC comments + regenerated proto
  • server/attachment_service: Fix "save the blob" → "saves the blob" in godoc
  • web/i18n: Rename isValidateLocaleisValidLocale (grammatically incorrect predicate name)

Test plan

  • No logic changes — only comments, error strings, and a function rename
  • Proto files regenerated via buf generate
  • isValidLocale rename is consistent across all 3 call sites in i18n.ts

🤖 Generated with Claude Code

- store/db/mysql: "eroor" → "error" in DSN parse error message
- plugin/storage/s3: "presign put object" → "presign get object" in PresignGetObject
- plugin/idp/oauth2: "failed to new http request" → "failed to create http request"
- plugin/cron/parser: "are parse in" → "are parsed in" in comment
- proto/attachment_service: "a attachment" → "an attachment" (3 occurrences)
- server/attachment_service: "save the blob" → "saves the blob" in godoc
- web/i18n: rename isValidateLocale → isValidLocale

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@memoclaw memoclaw requested a review from a team as a code owner March 8, 2026 10:00
@johnnyjoygh johnnyjoygh merged commit 8f43e80 into usememos:main Mar 8, 2026
8 checks passed
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