Skip to content

fix(ci): push format changes as bot - #597

Merged
Suyunmeng merged 4 commits into
mainfrom
fix/ci-validate-formatting
Jul 21, 2026
Merged

fix(ci): push format changes as bot#597
Suyunmeng merged 4 commits into
mainfrom
fix/ci-validate-formatting

Conversation

@Suyunmeng

Copy link
Copy Markdown
Contributor

Summary / 摘要

  • Configure the format workflow checkout to authenticate Git pushes with BOT_TOKEN.

  • Preserve the existing automatic Prettier formatting and direct signed commit behavior on main.

  • Prevent the format workflow from using the default Actions token, which the main ruleset rejects for direct pushes.

  • This PR has breaking changes.

  • This PR changes public API, config, storage format, or migration behavior.

  • This PR requires corresponding changes in related repositories.

Related repository PRs / 关联仓库 PR:

  • OpenList:
  • OpenList-Docs:

Testing / 测试

  • pnpm dlx prettier@3.8.4 --config .prettierrc --check .github/workflows/format.yml
  • git diff origin/main...HEAD --check
  • Manual test / 手动测试: Reviewed all repository workflow branch-write paths and the active main ruleset with gh.

Checklist / 检查清单

  • I have read CONTRIBUTING.
  • I confirm this contribution follows the repository license, contribution policy, and code of conduct.
  • I have formatted the changed code with gofmt, go fmt, or prettier where applicable.
  • I have requested review from relevant maintainers or code owners where applicable.

AI Disclosure / AI 使用声明

  • This PR includes AI-assisted content.

Tools used / 使用工具:

  • Claude

Usage scope / 使用范围:

  • Code generation

  • Tests

  • Review assistance

  • I have reviewed and validated all AI-assisted content included in this PR.

  • I have ensured that all AI-assisted commits include Co-Authored-By attribution.

  • I can reproduce all AI-assisted content included in this PR without any AI tools.

Suyunmeng and others added 4 commits July 22, 2026 04:11
- check Prettier formatting before changes reach main
- remove the protected-branch auto-push workflow

Co-authored-by: Claude <81847+claude@users.noreply.github.com>
- preserve formatting on pushes to main
- submit formatting changes through a bot pull request

Co-authored-by: Claude <81847+claude@users.noreply.github.com>
- merge bot formatting pull requests into main
- delete merged formatting branches automatically

Co-authored-by: Claude <81847+claude@users.noreply.github.com>
- authenticate checkout with the bot token
- retain direct automated formatting commits on main

Co-authored-by: Claude <81847+claude@users.noreply.github.com>
@Suyunmeng
Suyunmeng merged commit 8865625 into main Jul 21, 2026
1 check passed
@Suyunmeng
Suyunmeng deleted the fix/ci-validate-formatting branch July 21, 2026 20:41
Suyunmeng added a commit that referenced this pull request Jul 21, 2026
- authenticate checkout with the bot token
- retain direct automated formatting commits on main

Co-authored-by: Claude <81847+claude@users.noreply.github.com>
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