Skip to content

chore(actions): serialize maintenance and use scoped stats token - #80

Merged
trvny merged 4 commits into
mainfrom
chore/actions-safety-20261109
Nov 9, 2026
Merged

trvny merged 4 commits into
mainfrom
chore/actions-safety-20261109

Conversation

@trvny

@trvny trvny commented Nov 9, 2026 •

Copy link
Copy Markdown
Owner

Cancel stale pre-commit PR checks. Queue upkeep/settings operations without interrupting writes. Use GITHUB_TOKEN for public contributor collection; keep scoped PAT where cross-repo administration/gists require it.

View guided diff Turn on auto-fix

Co-authored-by: chatgpt-codex-connector[bot] <199175422+chatgpt-codex-connector[bot]@users.noreply.github.com>
@kanarek-companion

kanarek-companion Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

🐤 Kanarek · 🟣 merged

main ✅

Code is in the nest. The machinery may rest.

Kanarek · 4 files

@sourcery-ai

sourcery-ai Bot commented Nov 9, 2026

Copy link
Copy Markdown

Sorry @trvny, you've used your own review budget of 250,000 diff characters for the last 7 days.

You can request another review in 12 hours and 17 minutes by commenting @sourcery-ai review. Upgrade to get a review now.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-09T08:36:36.085443Z 10845b0 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@coderabbitai

coderabbitai Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

Explain Change →

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 93b6170a-0edf-4139-a5e3-236b29fcdf81
Walkthrough

改动涉及工作流并发配置和统计卡片令牌。pre-commit 按 ref 分组,并仅对 pull_request 事件取消同组运行。repo-settings-sync 和 upkeep 将并发队列上限设为 max,并关闭进行中运行取消。contributors card 步骤改用 github.token。

Possibly related PRs

  • trvny/.github#79:该 PR 修改同一 contributors card 步骤及其令牌所访问的 API 范围。
Merge Risk: Low · previous review 10845

The changes are mergeable with owner awareness: overlapping audits or PR updates can require rerunning checks, and shared API quota exhaustion can delay asset refreshes. Queue overflow also lacks reconciliation, although synchronization loss was already possible before this change.

Pre-merge checks: Passed 5

Check name Status Explanation
Title check Passed 标题 chore(actions): serialize maintenance and use scoped stats token 与变更相关。变更包括:为 pre-commit 工作流增加并发控制(PR 新运行取消旧运行);为 upkeep 和 repo-settings-sync 工作流加入 queue: max 并关闭 cancel-in-progress;stats 工作流的 cont…
Description check Passed 描述与变更相关。描述提到取消过期的 pre-commit PR 检查(对应 pre-commit.yml 的并发取消设置),将 upkeep 和设置同步操作排队以避免中断写入(对应 queue: max 与 cancel-in-progress 改为 false),并对公共贡献者统计改用 GITHUB_TOKEN,同时保留范围受限的 PAT 用于跨仓库管理和 gist(对应 stats.yml 的…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

标题 chore(actions): serialize maintenance and use scoped stats token 与变更相关。变更包括:为 pre-commit 工作流增加并发控制(PR 新运行取消旧运行);为 upkeep 和 repo-settings-sync 工作流加入 queue: max 并关闭 cancel-in-progress;stats 工作流的 contributors 步骤改用 github.token。标题中 serialize maintenance 对应队列化维护任务,use scoped stats token 对应 stats 令牌变更。标题覆盖主要变更,但未提及 pre-commit 取消过期运行这一项,因此属于部分相关,但仍是真实且具体的描述。标题为英文,符合 conventional commit 格式。

Full details: Description check

Explanation

描述与变更相关。描述提到取消过期的 pre-commit PR 检查(对应 pre-commit.yml 的并发取消设置),将 upkeep 和设置同步操作排队以避免中断写入(对应 queue: max 与 cancel-in-progress 改为 false),并对公共贡献者统计改用 GITHUB_TOKEN,同时保留范围受限的 PAT 用于跨仓库管理和 gist(对应 stats.yml 的变更)。描述与变更内容一致。

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.)

Finishing Touches

Simplify code

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

三条工作流,队列各有新规
PR 同 ref,旧运行可停
统计卡片,换用新令牌
并发设置,逐项落定
改动虽小,配置清晰

@mergify

mergify Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 4 files

View guided diff | Turn on auto-fix | Re-trigger cubic

Co-authored-by: chatgpt-codex-connector[bot] <199175422+chatgpt-codex-connector[bot]@users.noreply.github.com>

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 4

Comment thread .github/workflows/pre-commit.yml Resolved
Comment thread .github/workflows/pre-commit.yml Resolved
Comment thread .github/workflows/stats.yml Resolved
gptomek Bot and others added 2 commits November 9, 2026 08:47
Co-authored-by: chatgpt-codex-connector[bot] <199175422+chatgpt-codex-connector[bot]@users.noreply.github.com>
Co-authored-by: chatgpt-codex-connector[bot] <199175422+chatgpt-codex-connector[bot]@users.noreply.github.com>

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sourcery assessment

Approved.

@trvny
trvny merged commit 09b3bf7 into main Nov 9, 2026
9 checks passed
@trvny
trvny deleted the chore/actions-safety-20261109 branch November 9, 2026 09:00
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