Skip to content

ci: fix spelling action failure#926

Merged
ishymko merged 1 commit intomainfrom
spelling-main
Apr 2, 2026
Merged

ci: fix spelling action failure#926
ishymko merged 1 commit intomainfrom
spelling-main

Conversation

@ishymko
Copy link
Copy Markdown
Member

@ishymko ishymko commented Apr 2, 2026

See #922, pull_request_target apparently requires pushing to main.

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

There is [code in spelling
check](https://github.com/check-spelling/check-spelling/blob/c635c2f3f714eec2fcf27b643a1919b9a811ef2e/unknown-words.sh#L182)
which probes branch renaming to make sure that pipeline is set up
according to the
[guide](https://docs.check-spelling.dev/Feature:-Restricted-Permissions).

It doesn't specify request body and looks for a specific message in the
response assuming that token check happens before body validation. Seems
like GitHub change behavior and body schema is validated before
permissions, so such request fails with a different error now:

```
curl -s -X POST \
  -H "Accept: application/vnd.github+json" \
  -H "Authorization: Bearer $GITHUB_TOKEN" \
  "https://api.github.com/repos/a2aproject/a2a-python/branches/1.0-dev/rename"

{
  "message": "Invalid request.\n\nFor 'links/0/schema', nil is not an object.",
  "documentation_url": "https://docs.github.com/rest/branches/branches#rename-a-branch",
  "status": "422"
}
```

Providing body gives a proper error:

```json
{
  "message": "Resource not accessible by personal access token",
  "documentation_url": "https://docs.github.com/rest/branches/branches#rename-a-branch",
  "status": "403"
}
```

Trying to update the workflow according to the sample
https://raw.githubusercontent.com/check-spelling/spell-check-this/main/.github/workflows/spelling.yml.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 2, 2026

🧪 Code Coverage (vs main)

⬇️ Download Full Report

No coverage changes.

Generated by coverage-comment.yml

@ishymko ishymko marked this pull request as ready for review April 2, 2026 12:33
@ishymko ishymko requested a review from a team as a code owner April 2, 2026 12:33
@ishymko ishymko changed the title ci: fix spelling action failure (#922) ci: fix spelling action failure Apr 2, 2026
@ishymko ishymko merged commit a8eced8 into main Apr 2, 2026
14 checks passed
@ishymko ishymko deleted the spelling-main branch April 2, 2026 12:35
@ishymko
Copy link
Copy Markdown
Member Author

ishymko commented Apr 2, 2026

Didn't help, reported in check-spelling/check-spelling#103.

@jsoref
Copy link
Copy Markdown

jsoref commented Apr 3, 2026

Thanks. Sorry.

ishymko added a commit that referenced this pull request Apr 3, 2026
ishymko added a commit that referenced this pull request Apr 3, 2026
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