-
Notifications
You must be signed in to change notification settings - Fork 218
Added instructions for sdk local workflow #11868
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
raych1
commented
Aug 28, 2025
- this instruction file applies for the local development scenario
- currently includes the generate sdk and build sdk steps
- removed the old instruction file specially for python
There was a problem hiding this 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 the local SDK development workflow instructions by replacing the Python-specific file with a new general workflow that supports multiple languages. The changes consolidate and modernize the local SDK generation process.
- Replaces Python-only instructions with multi-language support (.NET, Java, JavaScript/TypeScript, Python, Go)
- Updates reference to point to the new general workflow file
- Removes outdated Python-specific instruction file
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| typespec-to-sdk.instructions.md | Updates file reference from create-sdk-locally to local-sdk-workflow |
| local-sdk-workflow.instructions.md | New comprehensive workflow supporting 5 languages with SDK generation and build steps |
| create-sdk-locally.instructions.md | Removes old Python-only instruction file |
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
|
The following pipelines have been queued for testing: |
|
@praveenkuttappan @LibbaLawrence |
|
The following pipelines have been queued for testing: |
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/typespec-to-sdk.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
|
@raych1 it would be good to add people from the inner loop so they can review too as this will affect their flow |
|
The following pipelines have been queued for testing: |
|
The following pipelines have been queued for testing: |
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
eng/common/instructions/azsdk-tools/typespec-to-sdk.instructions.md
Outdated
Show resolved
Hide resolved
|
The following pipelines have been queued for testing: |
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
|
The following pipelines have been queued for testing: |
|
The following pipelines have been queued for testing: |
eng/common/instructions/azsdk-tools/local-sdk-workflow.instructions.md
Outdated
Show resolved
Hide resolved
|
The following pipelines have been queued for testing: |
…tions.md Co-authored-by: Mariana Rios Flores <[email protected]>
…tions.md Co-authored-by: Mariana Rios Flores <[email protected]>
…tions.md Co-authored-by: Mariana Rios Flores <[email protected]>
…tions.md Co-authored-by: Mariana Rios Flores <[email protected]>
…tions.md Co-authored-by: Mariana Rios Flores <[email protected]>
eeffe54 to
0ea84e0
Compare
|
The following pipelines have been queued for testing: |
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#11868 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: ray chen <[email protected]> Co-authored-by: Mariana Rios Flores <[email protected]>
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#11868 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: ray chen <[email protected]> Co-authored-by: Mariana Rios Flores <[email protected]>