Skip to content

为代码检查集成 reviewdog - #6692

Open
neveler wants to merge 1 commit into
HMCL-dev:mainfrom
neveler:reviewdog
Open

为代码检查集成 reviewdog#6692
neveler wants to merge 1 commit into
HMCL-dev:mainfrom
neveler:reviewdog

Conversation

@neveler

@neveler neveler commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

优化代码检查流程,通过 reviewdog 将检查结果直接体现在 PR 状态检查中。


https://github.com/reviewdog/reviewdog
https://github.com/reviewdog/action-setup

image image

@github-actions github-actions Bot added the 1+ label Aug 16, 2026
@neveler neveler changed the title Update check-codes.yml configuration 为代码检查集成 reviewdog Aug 16, 2026
@Glavo

Glavo commented Aug 16, 2026

Copy link
Copy Markdown
Member

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c7ef111911

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/check-codes.yml Outdated
@neveler
neveler marked this pull request as draft August 16, 2026 15:41
@neveler neveler changed the title 为代码检查集成 reviewdog 为 PR 增加自动发布 checkstyle 审查评论的功能 Aug 17, 2026
@github-actions github-actions Bot added 40+ and removed 1+ labels Aug 17, 2026
@github-actions github-actions Bot added 1+ and removed 40+ labels Aug 17, 2026
@neveler neveler changed the title 为 PR 增加自动发布 checkstyle 审查评论的功能 为代码检查集成 reviewdog Aug 17, 2026
@neveler
neveler marked this pull request as ready for review August 17, 2026 07:00
@Glavo

Glavo commented Aug 17, 2026

Copy link
Copy Markdown
Member

@codex review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e940f016e7

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

- '**.properties'

permissions:
checks: write

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Handle forked pull requests without write tokens

For pull requests submitted from forks, GitHub downgrades GITHUB_TOKEN write permissions to read-only, as documented under permissions for forked repositories. Consequently, the github-pr-check reporter cannot create the intended Check annotations for external contributors despite checks: write; publish the reports from a trusted follow-up such as a workflow_run workflow rather than relying on this token in the untrusted PR run.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants