Skip to content

Remove direct PyPI check from isolated build - #8593

Merged
Amaury Chamayou (achamayou) merged 1 commit into
microsoft:mainfrom
shivasathwikb:user/sbasavapatna/remove-pypi-preflight
Oct 18, 2026
Merged

Amaury Chamayou (achamayou) merged 1 commit into
microsoft:mainfrom
shivasathwikb:user/sbasavapatna/remove-pypi-preflight

Conversation

@shivasathwikb

Copy link
Copy Markdown
Contributor

Motivation

The OneBranch PyPI release pipeline directly queries pypi.org before building the wheel to check whether the package version already exists.

OneBranch network isolation blocks direct access to pypi.org, causing valid tag runs to fail with:

curl: (7) Failed to connect to pypi.org port 443

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.

🟢 Approval recommended

The focused removal addresses the blocked request without changing the remaining package validation or publication flow.

0 open findings

What changed in this PR

Removes the direct PyPI preflight request so OneBranch network isolation no longer blocks valid release builds.

Changes:

  • Removes the package-version lookup while preserving wheel building and validation.
  • Updates documentation to describe publication-time overwrite protection.

Custom instructions used

Repository Copilot instructions supplied in the review context; no additional instruction files loaded.

File Description
.pipelines/​scripts/​build-pypi-package.sh Removes the direct PyPI version check.
.pipelines/​README.md Updates validation and overwrite-protection documentation.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@shivasathwikb Shiva Sathwik Basavapatna Ramaprasad (shivasathwikb) changed the title Remove direct PyPI release preflight Remove direct PyPI check from isolated build Oct 16, 2026
@achamayou
Amaury Chamayou (achamayou) merged commit df5cf34 into microsoft:main Oct 18, 2026
12 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.

3 participants