Remove servicing PR template from Copilot instructions#37761
Conversation
Co-authored-by: AndriySvyryd <6539701+AndriySvyryd@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This pull request removes the servicing PR description template from the Copilot instructions file. The template previously provided a structured format for PRs targeting release branches, including sections for Description, Customer impact, How found, Regression, Testing, and Risk. The removal simplifies the PR Guidelines section to focus solely on the instruction to target the main branch for new PRs.
Changes:
- Removed the servicing PR template from
.github/copilot-instructions.md - Simplified the Pull Request Guidelines section
roji
left a comment
roji
left a comment
There was a problem hiding this comment.
@AndriySvyryd do we want to find an alternative before removing this?
Unfortunately there doesn't seem to be a way to conditionally include instructions ("lazy load") into copilot-instructions.md (e.g. based on the target branch). We could maybe try hacking this via a skill (which itself is lazy loaded) but that's getting convoluted and possibly brittle.
We can go with the approach you proposed of having a traditional workflow to post the comment with the template, if you want...
I'll try adding a skill. I'm afraid that CCA would ignore comments posted by a bot |
|
Yeah, in any case I agree that a skill here is better than having the template in the instructions... |
The servicing PR description template (with Description, Customer impact, How found, Regression, Testing, Risk sections) has been removed from
.github/copilot-instructions.md. The PR Guidelines section now only retains the instruction to targetmainfor new PRs.Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.