Add applyTo frontmatter to copilot-instructions.md for TypeSpec file reviews#3
Add applyTo frontmatter to copilot-instructions.md for TypeSpec file reviews#3
Conversation
Co-authored-by: raosuhas <11067581+raosuhas@users.noreply.github.com>
|
@copilot when making a PR into a branch , is the copilot-instructions.md from the source branch used or is it used from the target branch ? |
When making a PR, the This is the expected behavior because:
In the case of PR #1, since |
ARM (Control Plane) API Specification Update Pull Request
Tip
Overwhelmed by all this guidance? See the
Getting helpsection at the bottom of this PR description.PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.
Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!
Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.
Description
The instruction system uses YAML frontmatter with
applyToto map instruction files to file patterns.copilot-instructions.mdlacked this frontmatter, preventing it from being applied during TypeSpec file reviews.Changes
.github/copilot-instructions.md:This mirrors the pattern used in
openapi-review.instructions.md(applyTo: "**/*.json"), ensuring TypeSpec-specific guidance is now automatically loaded when reviewing.tspfiles.Additional information
Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the
Generated ApiViewcomment added to this PR. You can use ApiView to show API versions diff.Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.
Getting help
Purpose of this PRandDue diligence checklist.write accessper aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositoriesNext Steps to Mergecomment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.and https://aka.ms/ci-fix.
queuedstate, please add a comment with contents/azp run.This should result in a new comment denoting a
PR validation pipelinehas started and the checks should be updated after few minutes.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.