Skip to content

chore(github): try to have productive copilot reviews#12780

Merged
ChristophWurst merged 1 commit intomainfrom
chore/github/productive-copilot-reviews
Apr 21, 2026
Merged

chore(github): try to have productive copilot reviews#12780
ChristophWurst merged 1 commit intomainfrom
chore/github/productive-copilot-reviews

Conversation

@ChristophWurst
Copy link
Copy Markdown
Member

@ChristophWurst ChristophWurst commented Apr 20, 2026

I'm trying to find a way out of the AI review churn seen in #9347 and #12654.

This comment was marked as outdated.

Signed-off-by: Christoph Wurst <1374172+ChristophWurst@users.noreply.github.com>
Copy link
Copy Markdown

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.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


## Strict exclusions — do not comment on these
- Code style, formatting, or whitespace
- Naming conventions (variables, functions, classes, files)
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.

Why not?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I'd leave that to us humans

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We can see how it goes and adjust the doc. This is just a start

@ChristophWurst ChristophWurst merged commit c7551e9 into main Apr 21, 2026
46 checks passed
@ChristophWurst ChristophWurst deleted the chore/github/productive-copilot-reviews branch April 21, 2026 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants