Skip to content

fix: use idiomatic "at most" in English max/lte messages - #1600

Merged
nodivbyzero merged 1 commit into
go-playground:masterfrom
snowyukitty:fix/en-at-most-wording
Jul 29, 2026
Merged

fix: use idiomatic "at most" in English max/lte messages#1600
nodivbyzero merged 1 commit into
go-playground:masterfrom
snowyukitty:fix/en-at-most-wording

Conversation

@snowyukitty

Copy link
Copy Markdown
Contributor

Fixes Or Enhances

Fixes #1249.

English defaults for max-items, lte-items, and lte-string used "at maximum", which is unidiomatic. They now use "at most", matching the existing "at least" wording used by min/gte and the language already used in doc.go for max string length ("at most").

max-string already uses the correct phrase "a maximum of" and is left unchanged.

The same sibling scope (max-items together with related upper-bound item messages) has landed before in translation work such as #1250 for Japanese.

Make sure that you've checked the boxes below before you submit PR:

  • Tests exist or have been written that cover this particular change.

Validation:

  • On base 53a4402, updated expectations alone fail because the library still emits "at maximum"
  • go test ./translations/en passes on this branch
  • CGO_ENABLED=0 go test -cover ./... passes (root package coverage 95.2%; -race not run on this host for lack of CGO)

@go-playground/validator-maintainers

Fixes go-playground#1249.

The English defaults for max-items, lte-items, and lte-string said
"at maximum", which is unidiomatic. Align them with the existing
"at least" min/gte wording and with doc.go, which already describes
max string length as "at most".

Leave max-string as "a maximum of", which is already correct English.
@snowyukitty
snowyukitty requested a review from a team as a code owner July 18, 2026 22:51
@nodivbyzero
nodivbyzero merged commit f9944c5 into go-playground:master Jul 29, 2026
7 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.

en translation issue: "must contain at maximum X items"

3 participants