Skip to content

Add balance and transfer context to copilot instructions#444

Merged
ovitrif merged 2 commits into
masterfrom
chore/balance-transfer-instructions-review
Oct 21, 2025
Merged

Add balance and transfer context to copilot instructions#444
ovitrif merged 2 commits into
masterfrom
chore/balance-transfer-instructions-review

Conversation

@jvsena42
Copy link
Copy Markdown
Member

Description

Improve copilot instructions with transfer and balance context

Preview

N/A

QA Notes

N/A

@jvsena42 jvsena42 requested review from Copilot and ovitrif October 21, 2025 14:10
@jvsena42 jvsena42 self-assigned this Oct 21, 2025
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 enhances GitHub Copilot's code review instructions by adding references to domain-specific documentation for balance management and transfer operations.

Key Changes:

  • Added a new "Business Logic" section to copilot instructions
  • Introduced context directives linking to balance and transfer documentation

Comment thread .github/copilot-instructions.md Outdated
Copy link
Copy Markdown
Collaborator

@ovitrif ovitrif left a comment

Choose a reason for hiding this comment

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

Maybe we can also try to be more generalistic, just asking to review against *.md files from docs/ or something new like docs/ai/, see how that goes 🙃

@jvsena42
Copy link
Copy Markdown
Member Author

Maybe we can also try to be more generalistic, just asking to review against *.md files from docs/ or something new like docs/ai/, see how that goes 🙃

I thought the same, but implemented like this because the doc only used files as reference
https://docs.github.com/en/copilot/how-tos/use-copilot-agents/request-a-code-review/use-code-review#customizing-copilots-reviews-with-custom-instructions

@ovitrif
Copy link
Copy Markdown
Collaborator

ovitrif commented Oct 21, 2025

Maybe we can also try to be more generalistic, just asking to review against *.md files from docs/ or something new like docs/ai/, see how that goes 🙃

I thought the same, but implemented like this because the doc only used files as reference
https://docs.github.com/en/copilot/how-tos/use-copilot-agents/request-a-code-review/use-code-review#customizing-copilots-reviews-with-custom-instructions

Oh, very nice then 🙌🏻
Let's go 🚀🚀🚀

@ovitrif ovitrif merged commit 05c3bfe into master Oct 21, 2025
10 of 11 checks passed
@ovitrif ovitrif deleted the chore/balance-transfer-instructions-review branch October 21, 2025 14:34
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