Skip to content

feat: relax requests lower bound to >=2.31.0 for MWAA compatibility#63

Merged
zongsizhang merged 2 commits intomainfrom
relax-requests-dep
Mar 23, 2026
Merged

feat: relax requests lower bound to >=2.31.0 for MWAA compatibility#63
zongsizhang merged 2 commits intomainfrom
relax-requests-dep

Conversation

@zongsizhang
Copy link
Copy Markdown
Contributor

Summary

  • Lowered requests minimum version from >=2.32.3 to >=2.31.0 in pyproject.toml
  • MWAA environments may ship with requests 2.31.x, which is incompatible with the previous >=2.32.3 constraint

Test plan

  • Verify package installs correctly with requests==2.31.0
  • Run existing test suite to confirm no regressions

🤖 Generated with Claude Code

Lower the requests minimum version from 2.32.3 to 2.31.0 to support
MWAA environments that may ship with older requests versions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@zongsizhang zongsizhang requested a review from a team as a code owner March 23, 2026 20:42
@zongsizhang zongsizhang requested review from haizhou-zhao and removed request for a team March 23, 2026 20:42
Copy link
Copy Markdown

@claude claude bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review.

Tip: disable this comment in your organization's Code Review settings.

Copy link
Copy Markdown

@salty-hambot salty-hambot bot left a comment

Choose a reason for hiding this comment

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

Reviewed by Salty Hambot 🤖🧂

Surgical change, one loose thread: the types-requests stubs floor (2.32.x) now outpaces the runtime floor (2.31.0) — worth confirming no API drift lurks in that gap.

1 finding(s) posted.
💬 To request a re-review, comment @salty-hambot review

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@zongsizhang zongsizhang merged commit a9a8394 into main Mar 23, 2026
7 checks passed
@zongsizhang zongsizhang deleted the relax-requests-dep branch March 23, 2026 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants