Skip to content

chore: add Copilot review instuctions#1507

Merged
fi3ework merged 1 commit intomainfrom
copilot-c
Mar 3, 2026
Merged

chore: add Copilot review instuctions#1507
fi3ework merged 1 commit intomainfrom
copilot-c

Conversation

@fi3ework
Copy link
Copy Markdown
Member

@fi3ework fi3ework commented Mar 3, 2026

Summary

i had the agent collect all non-BOT human review comments from the past two months and then summarized rule instructions applicable to future PRs, hoping this will reduce the burden of manual review and make Copilot's review information less verbose.

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings March 3, 2026 06: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

Adds a repository-level set of Copilot code review instructions distilled from recent human review feedback, aiming to reduce review burden and verbosity.

Changes:

  • Introduces general PR review rules focused on goal alignment and limiting blast radius.
  • Adds documentation-specific review rules for Markdown/MDX and user-facing guides.
Comments suppressed due to low confidence (1)

.github/copilot-instructions.md:30

  • The link requirements in lines 25 and 30 overlap and partially duplicate each other, which conflicts with the rule on removing duplication (line 28). Consider merging these into a single bullet that covers both config/API links and package/tool links, to keep the instruction set concise and non-redundant.
- Link each mentioned config option or API to its canonical official documentation.
- Call out global side effects of configuration choices and propose safer scoped alternatives when relevant.
- Avoid misleading examples or outputs; add clarifying context when artifacts may look incorrect.
- Remove duplicated explanations and keep one authoritative location per concept.
- Prefer concise, copy-pasteable command examples when multiple dependencies or setup steps are required.
- Ensure every mentioned package, plugin, or tool has a direct authoritative reference link.

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

@fi3ework fi3ework merged commit 61f0061 into main Mar 3, 2026
16 checks passed
@fi3ework fi3ework deleted the copilot-c branch March 3, 2026 06:50
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