Skip to content

Revert "Update @typespec/http-client-python to 0.35.0" (DO NOT MERGE) - #5009

Draft
msyyc wants to merge 1 commit into
mainfrom
revert-5003-l0lawrence-cuddly-invention
Draft

Revert "Update @typespec/http-client-python to 0.35.0" (DO NOT MERGE)#5009
msyyc wants to merge 1 commit into
mainfrom
revert-5003-l0lawrence-cuddly-invention

Conversation

@msyyc

@msyyc msyyc commented Jul 22, 2026

Copy link
Copy Markdown
Member

Only for test and do not merge

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

No changes needing a change description found.

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

You can try these changes here

🛝 Playground 🌐 Website

XiaofeiCao pushed a commit to XiaofeiCao/typespec-azure that referenced this pull request Jul 28, 2026
…zure#5059)

Fixes Azure#5010.

## Problem

`@typespec/http-client-python` is not a workspace package — it is
consumed from npm through the pnpm **catalog**, so bumping it only edits
the root `pnpm-workspace.yaml`.

pnpm attributes root-file changes to the **root package only**, so `pnpm
--filter "...[base]"` reports no affected emitter and `detect-affected`
returns `python: false`. That is why Azure#5009 ran no Python CI.

## Fix

Add `pnpm-workspace.yaml` to `sharedExtra`, so any change to it triggers
every emitter target — the same treatment the `core` submodule pointer
already gets.

The file changes rarely and always affects dependency resolution
repo-wide, so this is deliberately blunt rather than trying to work out
which catalog entry moved.

## Verification

Simulated the Azure#5009 diff locally with `BASE_SHA=origin/main`:

```
Affected targets: {"python":true,"java":true,"typescript":true,"go":true}
```

Plus a unit test in `eng/scripts/detect-affected.test.ts`.

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 19d6a84d-f603-41ee-8377-6dfb7e661391
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.

1 participant