Skip to content

Conversation

cheenamalhotra
Copy link
Member

Add more elaborate instructions.

@Copilot Copilot AI review requested due to automatic review settings June 17, 2025 21:56
@cheenamalhotra cheenamalhotra requested a review from a team as a code owner June 17, 2025 21:56
Copy link
Contributor

@Copilot 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 introduces a new copilot-instructions.md file under .github/ to document roles, workflows, and best practices for triaging issues, fixing bugs, writing tests, creating PRs, and automating processes within the dotnet/SqlClient repository.

  • Defines responsibilities (triage, fixing, testing, documentation, reviews)
  • Outlines step-by-step workflows (issues → bugs → PRs → closures)
  • Adds contextual guidance and constraints for contributors and automation
Comments suppressed due to low confidence (2)

.github/copilot-instructions.md:130

  • This suggests updating CHANGELOG.md, but later constraints prohibit modifying it directly. Clarify the intended workflow for changelog updates to avoid confusion.
- Suggest changelog entries for fixes in `CHANGELOG.md`

.github/copilot-instructions.md:19

  • [nitpick] The nested list item here uses two spaces before the dash, while other nested lists use consistent indentation. Align this with the rest of the markdown for readability.
  - **Unit Tests**: Located in `tests/UnitTests/` directory, which includes tests for individual components and methods.

@cheenamalhotra cheenamalhotra merged commit 5ded2e5 into main Jun 18, 2025
6 checks passed
@cheenamalhotra cheenamalhotra deleted the dev/cheena/copilot branch June 18, 2025 22:34
@paulmedynski paulmedynski added this to the 6.1-preview2 milestone Jun 23, 2025
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