Skip to content

Move Storage Extension to Use Pyproject.toml#47351

Open
kashifkhan wants to merge 2 commits into
Azure:mainfrom
kashifkhan:move_search_ext_to_pyproject
Open

Move Storage Extension to Use Pyproject.toml#47351
kashifkhan wants to merge 2 commits into
Azure:mainfrom
kashifkhan:move_search_ext_to_pyproject

Conversation

@kashifkhan
Copy link
Copy Markdown
Member

@kashifkhan kashifkhan commented Jun 4, 2026

Moving storage extensions to use pyproject.toml. setup.py remains since building extensions is marked as experiment for pyproject.toml

Copilot AI review requested due to automatic review settings June 4, 2026 15:23
@github-actions github-actions Bot added the Storage Storage Service (Queues, Blobs, Files) label Jun 4, 2026
Copy link
Copy Markdown
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 migrates the azure-storage-extensions package’s packaging metadata from setup.py into pyproject.toml, while keeping a minimal setup.py to support building the C extension and producing abi3-tagged wheels.

Changes:

  • Added full PEP 621 project/build metadata to pyproject.toml (including dynamic version/readme configuration).
  • Simplified setup.py to only define the limited-API C extension and override bdist_wheel tagging to abi3.
  • Added [tool.azure-sdk-conda] metadata for the package.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
sdk/storage/azure-storage-extensions/setup.py Reduced to only extension build + bdist_wheel abi3 tag override, relying on pyproject.toml for metadata.
sdk/storage/azure-storage-extensions/pyproject.toml Introduces build-system + project metadata and setuptools dynamic configuration, plus conda metadata.

Comment thread sdk/storage/azure-storage-extensions/pyproject.toml
]

[tool.setuptools.package-data]
"azure.storage.extensions.checksums" = ["py.typed", "*.pyi"]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be needed as I believe setuptools picks these by default. You can try removing and checking the wheel contents

@kashifkhan kashifkhan changed the title Move Search Extension to Use Pyproject.toml Move Storage Extension to Use Pyproject.toml Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Storage Storage Service (Queues, Blobs, Files)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants