Skip to content

[devex agent] TypeSpec that is merged can remain on main branch. #11701

Merged
jeo02 merged 1 commit intoAzure:mainfrom
jeo02:typespec-merged-stay-on-main
Aug 18, 2025
Merged

[devex agent] TypeSpec that is merged can remain on main branch. #11701
jeo02 merged 1 commit intoAzure:mainfrom
jeo02:typespec-merged-stay-on-main

Conversation

@jeo02
Copy link
Member

@jeo02 jeo02 commented Aug 18, 2025

Removed pre flight check and added in a extra step to make a branch inside of the commit changes step.

Copilot AI review requested due to automatic review settings August 18, 2025 18:26
@jeo02 jeo02 requested a review from a team as a code owner August 18, 2025 18:26
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 the TypeSpec to SDK generation instructions by removing the pre-flight check for branch verification and moving the branch check to occur during the commit phase instead. This allows TypeSpec projects to remain on the main branch during development and only requires a new branch when committing changes.

  • Removed the "Pre-Flight Check" section that verified the user was not on the main branch at the start
  • Added branch verification step within the commit changes workflow
  • Moved user prompting for branch creation to the point where changes are actually committed

@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 added a commit to Azure/azure-sdk-for-js that referenced this pull request Aug 18, 2025
Sync eng/common directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#11701 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

Co-authored-by: Juan Ospina <70209456+jeo02@users.noreply.github.com>
azure-sdk added a commit to Azure/azure-sdk-for-rust that referenced this pull request Aug 18, 2025
Sync eng/common directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#11701 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

Co-authored-by: Juan Ospina <70209456+jeo02@users.noreply.github.com>
@jeo02 jeo02 merged commit 15f7cc7 into Azure:main Aug 18, 2025
9 checks passed
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.

4 participants