Skip to content

Add copilot-instructions#4890

Merged
sisuresh merged 1 commit intostellar:masterfrom
sisuresh:add-ins
Aug 26, 2025
Merged

Add copilot-instructions#4890
sisuresh merged 1 commit intostellar:masterfrom
sisuresh:add-ins

Conversation

@sisuresh
Copy link
Contributor

@sisuresh sisuresh commented Aug 20, 2025

Description

Adds copilot instructions to comment if fee bumps tests should be written.

We could also make additional changes for more targeted reviews on things like style guidelines and missing tests.

Here's an example of this in action - sisuresh#15 (comment)

Checklist

  • Reviewed the contributing document
  • Rebased on top of master (no merge commits)
  • Ran clang-format v8.0.0 (via make format or the Visual Studio extension)
  • Compiles
  • Ran all tests
  • If change impacts performance, include supporting evidence per the performance document

Copilot AI review requested due to automatic review settings August 20, 2025 22:08
Copy link

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 adds GitHub Copilot instructions to the repository to help improve code review quality by providing specific guidance on test coverage requirements. The instructions focus on ensuring that fee bump tests are written alongside regular transaction tests.

  • Adds .github/copilot-instructions.md file with fee bump testing guidelines
  • Provides automated guidance for code reviewers about test coverage expectations

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sisuresh sisuresh requested a review from dmkozh August 20, 2025 23:31
@sisuresh sisuresh enabled auto-merge August 26, 2025 19:59
@sisuresh sisuresh added this pull request to the merge queue Aug 26, 2025
Merged via the queue into stellar:master with commit 2673afa Aug 26, 2025
16 checks passed
@sisuresh sisuresh deleted the add-ins branch August 26, 2025 20:40
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.

2 participants

Comments