Skip to content

Fix/doc pr eyes no ack#922

Merged
jth-nw merged 2 commits into
devfrom
fix/doc-pr-eyes-no-ack
May 14, 2026
Merged

Fix/doc pr eyes no ack#922
jth-nw merged 2 commits into
devfrom
fix/doc-pr-eyes-no-ack

Conversation

@jth-nw
Copy link
Copy Markdown
Collaborator

@jth-nw jth-nw commented May 14, 2026

No description provided.

jth-nw and others added 2 commits May 14, 2026 10:21
The ack comment added noise, especially for questions. Replace it with
a 👀 reaction on the triggering comment (immediate, unobtrusive feedback)
and have Claude post its own comments: a direct answer for questions,
or a progress checklist for file-edit tasks.

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
…be posted

VALE_TOKEN only has contents:write scope and cannot post or edit PR
comments (403). The action overrides GH_TOKEN with whatever github_token
is set to, so all gh CLI calls inherited the wrong token. Switching to
GITHUB_TOKEN fixes comment posting. Git pushes still use VALE_TOKEN
because actions/checkout already configured the git credential helper
with it — the two tokens are independent.

Generated with AI

Co-Authored-By: Claude Code <ai@netwrix.com>
@jth-nw jth-nw merged commit 8f72f88 into dev May 14, 2026
3 of 6 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.

1 participant