Skip to content

replace ack comment with eyes emoji reaction#923

Merged
jth-nw merged 3 commits into
mainfrom
dev
May 14, 2026
Merged

replace ack comment with eyes emoji reaction#923
jth-nw merged 3 commits into
mainfrom
dev

Conversation

@jth-nw
Copy link
Copy Markdown
Collaborator

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

No description provided.

jth-nw and others added 3 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 50b1697 into main May 14, 2026
5 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