Skip to content

Remove dependency on persistent token#13474

Merged
williammartin merged 1 commit into
trunkfrom
wm-no-more-automation-token
May 20, 2026
Merged

Remove dependency on persistent token#13474
williammartin merged 1 commit into
trunkfrom
wm-no-more-automation-token

Conversation

@williammartin

@williammartin williammartin commented May 20, 2026

Copy link
Copy Markdown
Member

Description

This token used to be used for lots more things and had a lot more privilege than necessary. Think it can just go away in favour of an actions token.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the spam-issue detection workflow to stop depending on a long-lived automation token and instead use the workflow-issued GITHUB_TOKEN.

Changes:

  • Removes the job’s dependency on the cli-automation environment (and its secret token).
  • Switches GH_TOKEN from secrets.AUTOMATION_TOKEN to ${{ github.token }}.
  • Adds inline documentation describing the intended minimal GITHUB_TOKEN permissions for the workflow.
Show a summary per file
File Description
.github/workflows/detect-spam.yml Replace persistent token usage with GITHUB_TOKEN and document required permissions.

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 .github/workflows/detect-spam.yml Outdated
Comment thread .github/workflows/detect-spam.yml Outdated
@williammartin williammartin force-pushed the wm-no-more-automation-token branch 2 times, most recently from f63bb0b to 309417e Compare May 20, 2026 18:45
@williammartin williammartin force-pushed the wm-no-more-automation-token branch from 309417e to 57480dd Compare May 20, 2026 18:48
@williammartin williammartin enabled auto-merge May 20, 2026 18:49
@williammartin williammartin merged commit 2fb8566 into trunk May 20, 2026
9 checks passed
@williammartin williammartin deleted the wm-no-more-automation-token branch May 20, 2026 18:56
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