Skip to content

Conversation

@jeo02
Copy link
Member

@jeo02 jeo02 commented Aug 26, 2025

Summary

  • Changed the tool names to the new snake case naming convention

Copilot AI review requested due to automatic review settings August 26, 2025 19:51
@jeo02 jeo02 requested a review from a team as a code owner August 26, 2025 19:51
Copy link
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 updates instruction files to align with new snake_case naming conventions for Azure SDK tools. The changes replace camelCase tool names with their snake_case equivalents across multiple instruction files.

Key Changes

  • Updated all tool names from camelCase to snake_case format (e.g., LinkNameSpaceApprovalIssueazsdk_link_namespace_approval_issue)
  • Corrected a filename from .instruction.md to .instructions.md for consistency

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
verify-namespace-approval.instructions.md Updated tool name for linking namespace approval issues
typespec-to-sdk.instructions.md Updated multiple tool names for GitHub operations and SDK generation
sdk-details-in-release-plan.instructions.md Updated tool name for release plan SDK information updates
create-release-plan.instructions.md Updated tool names for release plan creation and management
check-package-readiness.instructions.md Updated tool name for package readiness checks

@jeo02 jeo02 closed this Aug 26, 2025
@jeo02 jeo02 reopened this Aug 26, 2025
@azure-sdk
Copy link
Collaborator

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

@azure-sdk
Copy link
Collaborator

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

@weshaggard
Copy link
Member

For my context is there a reason for the tool name convention change?

@jeo02
Copy link
Member Author

jeo02 commented Aug 26, 2025

For my context is there a reason for the tool name convention change?

In this PR #11619 we changed all the names to the tools. Subsequently it changes how the tools are referred to by copilot. To make sure we are referring to the correct tool and not just copilots guess of what tool sounds similar enough we need to change the naming in the instructions as well.

azure-sdk added a commit to Azure/azure-sdk-for-js that referenced this pull request Aug 26, 2025
azure-sdk added a commit to Azure/azure-sdk-for-rust that referenced this pull request Aug 26, 2025
@jeo02 jeo02 merged commit f48e14b into Azure:main Aug 26, 2025
7 checks passed
@benbp
Copy link
Member

benbp commented Aug 27, 2025

@maririos @jeo02 is it worth thinking about generating these instructions from templates, that way we can inject things that need to reference named properties? Or we can plug it into the tool selection tests once those get spun up.

danieljurek pushed a commit that referenced this pull request Aug 29, 2025
* snake case

* change file name to .instructions.md

* Revert "change file name to .instructions.md"

This reverts commit c4bb65d.
radhgupta pushed a commit to radhgupta/azure-sdk-tools that referenced this pull request Sep 9, 2025
* snake case

* change file name to .instructions.md

* Revert "change file name to .instructions.md"

This reverts commit c4bb65d.
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.

6 participants