Skip to content

使用简体中文进行Copilot Code Review#731

Merged
CodFrm merged 6 commits intoscriptscat:mainfrom
cyfung1031:auto-Copilot
Sep 14, 2025
Merged

使用简体中文进行Copilot Code Review#731
CodFrm merged 6 commits intoscriptscat:mainfrom
cyfung1031:auto-Copilot

Conversation

@cyfung1031
Copy link
Copy Markdown
Collaborator

@cyfung1031 cyfung1031 commented Sep 13, 2025

有兩種方式。一是改 pull_request_template.md 一是改 copilot-instructions.md
先試改 pull_request_template.md

@CodFrm
Copy link
Copy Markdown
Member

CodFrm commented Sep 14, 2025

我测试了一下,这是有作用的 👍

@CodFrm
Copy link
Copy Markdown
Member

CodFrm commented Sep 14, 2025

https://docs.github.com/en/copilot/how-tos/use-copilot-agents/request-a-code-review/use-code-review#example

不过似乎修改 copilot-instructions.md 也可以,在 pull_request_template.md 中添加,那些注释有些影响 PR 人员编写内容(不过模板主题内容可以保留)

@CodFrm CodFrm requested a review from Copilot September 14, 2025 03:23
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 modifies the pull request template to use Simplified Chinese, changing the language from English to Chinese for better localization. The template structure remains the same but with translated section headers and instructions.

Key Changes:

  • Translates all section headers and placeholder text to Simplified Chinese
  • Maintains the same template structure and functionality
  • Updates comment instructions to guide Chinese-speaking contributors

@CodFrm
Copy link
Copy Markdown
Member

CodFrm commented Sep 14, 2025

https://docs.github.com/en/copilot/how-tos/use-copilot-agents/request-a-code-review/use-code-review#example

不过似乎修改 copilot-instructions.md 也可以,在 pull_request_template.md 中添加,那些注释有些影响 PR 人员编写内容(不过模板主题内容可以保留)

我在其它仓库测试是可以的,难道是提示词太乱了?

@CodFrm CodFrm requested review from Copilot and removed request for Copilot September 14, 2025 03:46
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@CodFrm CodFrm requested a review from Copilot September 14, 2025 03:59
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@CodFrm
Copy link
Copy Markdown
Member

CodFrm commented Sep 14, 2025

CodFrm#2

可能需要新的PR,等后续再注意一下

@CodFrm CodFrm merged commit 674a9ed into scriptscat:main Sep 14, 2025
3 checks passed
@cyfung1031
Copy link
Copy Markdown
Collaborator Author

cyfung1031 commented Sep 15, 2025

pull_request_template.md 中最重要的部份被你刪掉了。。。。
算了你喜歡吧

<!-- for GitHub Copilot review rule -->
<!--
如果需要修正 PR 的标题,请指出
请用簡體中文进行审查
-->
<!-- for GitHub Copilot review rule -->

@CodFrm
Copy link
Copy Markdown
Member

CodFrm commented Sep 16, 2025

@cyfung1031 这个原因我上面说了,#731 (comment)

@cyfung1031 cyfung1031 deleted the auto-Copilot branch October 4, 2025 02:09
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