Skip to content

Spanish spelling corrections#1331

Closed
jsoref wants to merge 2 commits into
github:mainfrom
jsoref:spelling-spanish
Closed

Spanish spelling corrections#1331
jsoref wants to merge 2 commits into
github:mainfrom
jsoref:spelling-spanish

Conversation

@jsoref
Copy link
Copy Markdown
Contributor

@jsoref jsoref commented May 31, 2026

  • Have you followed the contributing guidelines? -- they aren't really applicable, and this repository doesn't actually support issues.
  • Have you explained what your changes do, and why they add value?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.


Reported by check-spelling in https://github.com/check-spelling-sandbox/opensourcefriday/actions/runs/26723346400#summary-78754262966 using
https://github.com/check-spelling/cspell-dicts/blob/v20230509/dictionaries/es_ES/src/hunspell/index.dic
https://github.com/check-spelling/cspell-dicts/blob/v20230509/dictionaries/es_ES/src/hunspell/index.aff


Admittedly, I'm not a native Spanish speaker (I studied Spanish in High School decades ago...), but ...

jsoref added 2 commits May 31, 2026 16:36
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
@jsoref jsoref requested a review from a team as a code owner May 31, 2026 20:45
@jsoref
Copy link
Copy Markdown
Contributor Author

jsoref commented May 31, 2026

@AndreaGriffiths11 it looks like you have a required check for a workflow that doesn't trigger for pull requests...

@AndreaGriffiths11
Copy link
Copy Markdown
Contributor

Hey @jsoref — re-opening this from a local branch as #1332 so CodeQL runs (fork PRs in this repo can't satisfy the org code-scanning ruleset, which is why this PR stays BLOCKED even with green tests). You're preserved as author on both commits.

Heads up on one of them: the imponete → imponente change actually changed the meaning rather than fixing a typo. The original imponete límites claros para vos mismo is voseo (the second-person form used in Argentina, Uruguay, much of Central America), where imponete is the imperative of imponer + reflexive te — "set [for yourself]" — and pairs with para vos mismo. imponente is the adjective "imposing/impressive," which would have made the sentence ungrammatical. Thanks a lot for taking a look at this repo! Closing this in favor of #1332.

auto-merge was automatically disabled May 31, 2026 22:55

Pull request was closed

AndreaGriffiths11 added a commit that referenced this pull request May 31, 2026
Spanish spelling corrections (re-opened from #1331)
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