Skip to content

[Data Tables] Remove PyPy aiohttp version pinning - #49197

Merged
Kashif Khan (kashifkhan) merged 1 commit into
Azure:mainfrom
kashifkhan:update_data_tables_devreqs
Sep 25, 2026
Merged

Kashif Khan (kashifkhan) merged 1 commit into
Azure:mainfrom
kashifkhan:update_data_tables_devreqs

Conversation

@kashifkhan

Copy link
Copy Markdown
Member

We no longer have pypy in the ci, removing the lower bound aiohttp for it.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
10 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

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.

Copilot review overview

🟢 Approval recommended

The obsolete PyPy-specific constraint is removed cleanly with no remaining package references.

Review effort: Balanced
Findings: None

What changed in this PR

Removes obsolete PyPy-specific aiohttp dependency handling.

Changes:

  • Replaces conditional aiohttp requirements with one unpinned dependency.
  • Removes the obsolete PyPy workaround comment.
File Description
sdk/​tables/​azure-data-tables/​dev_requirements.txt Simplifies the development dependency on aiohttp.

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

@kashifkhan
Kashif Khan (kashifkhan) merged commit b47531c into Azure:main Sep 25, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants