Skip to content

Revert "Update TypeSpec Python regenerate workflow sync"#47884

Merged
msyyc merged 1 commit into
mainfrom
revert-47843-copilot/typespec-python-regenerate-main-sync
Jul 7, 2026
Merged

Revert "Update TypeSpec Python regenerate workflow sync"#47884
msyyc merged 1 commit into
mainfrom
revert-47843-copilot/typespec-python-regenerate-main-sync

Conversation

@msyyc

@msyyc msyyc commented Jul 7, 2026

Copy link
Copy Markdown
Member

Reverts #47843

This PR will introduce unexpected code diff which makes review difficult like #47883

Copilot AI review requested due to automatic review settings July 7, 2026 02:25
@msyyc msyyc requested review from a team, ChenxiJiang333, iscai-msft and tadelesh as code owners July 7, 2026 02:25

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 reverts PR #47843 ("Update TypeSpec Python regenerate workflow sync") in the typespec-python-regenerate.yml GitHub Actions workflow. Per the description, #47843 caused the automated regeneration to include unrelated main changes in its diff (by merging main into the generated-tests branch), making the resulting auto-generated PRs (e.g., #47883) harder to review. Reverting restores the simpler regeneration flow where the generated diff reflects only the TypeSpec-driven changes.

Changes:

  • Removes the merge-of-main-into-target-branch logic added by #47843, so main-based runs no longer pull in unrelated changes.
  • Restores the shallow --depth=1 fetch of the target branch.
  • Reverts the git-author identity setup from a single global config step back to inline local git config in the two steps that perform git merge and git commit.

@msyyc msyyc merged commit 43111bf into main Jul 7, 2026
23 checks passed
@msyyc msyyc deleted the revert-47843-copilot/typespec-python-regenerate-main-sync branch July 7, 2026 03:01
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.

3 participants