Skip to content

Spanish spelling corrections (re-opened from #1331)#1332

Merged
AndreaGriffiths11 merged 4 commits into
mainfrom
patch/es-spelling-0531-local
May 31, 2026
Merged

Spanish spelling corrections (re-opened from #1331)#1332
AndreaGriffiths11 merged 4 commits into
mainfrom
patch/es-spelling-0531-local

Conversation

@AndreaGriffiths11
Copy link
Copy Markdown
Contributor

@AndreaGriffiths11 AndreaGriffiths11 commented May 31, 2026

Re-opens the spelling fixes from #1331 from a local branch so CodeQL runs and the org code-scanning ruleset can be satisfied (fork PRs don't get CodeQL coverage in this repo, leaving #1331 permanently BLOCKED even with green tests).

Originally cherry-picked both of @jsoref's commits. After Copilot review I reverted one:

✅ Lider → Líder — kept (missing accent)
⛔ imponete → imponente — reverted. imponete is the voseo imperative of imponer + reflexive te (paired with vos mismo on the next line), not a typo for imponte. imponente is an unrelated adjective ("imposing"), so the swap would have changed the meaning.
Net change: one accent on Líder.

Closes #1331.

jsoref added 2 commits May 31, 2026 18:45
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 31, 2026 22:46
@AndreaGriffiths11 AndreaGriffiths11 requested a review from a team as a code owner May 31, 2026 22:46
Copy link
Copy Markdown
Contributor

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

Re-opens previously proposed Spanish locale text fixes so CodeQL can run under the repo’s code-scanning ruleset (fork PRs aren’t covered), targeting spelling in the Spanish translation strings.

Changes:

  • Updates the Spanish maintainer workflow “boundaries” description text.
  • Adds the accent to “Líder” in the Spanish endorsement title for Chris Lamb.
Show a summary per file
File Description
_data/locales/es.yml Adjusts Spanish locale strings (boundaries description + endorsement title spelling).

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 1

Comment thread _data/locales/es.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@AndreaGriffiths11 AndreaGriffiths11 merged commit 28f6cd1 into main May 31, 2026
5 checks passed
@AndreaGriffiths11 AndreaGriffiths11 deleted the patch/es-spelling-0531-local branch May 31, 2026 23:45
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