Skip to content

chore(deps): apply dependabot dependency bumps - #343

Open
jplbrun wants to merge 2 commits into
mainfrom
chore/dependabot-dep-bumps
Open

jplbrun wants to merge 2 commits into
mainfrom
chore/dependabot-dep-bumps

Conversation

@jplbrun

@jplbrun jplbrun commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Disclaimer: Do not include SAP-internal or customer-specific information in this PR (e.g. internal system URLs, customer names, tenant IDs, or confidential configurations). This is a public repository.

Description

Applies the currently-open Dependabot dependency updates in a single branch:

Dependency Change Dependabot PR
minio >=7.2.16,<7.2.17>=7.2.16,<7.2.21 #339
setuptools >=83.0,<84>=83.0,<85 #310
django (optional extra) >=4.0,<5>=4.0,<6 #305 / #312
ty (dev) 0.0.640.0.80 #338
ruff (dev) 0.16.00.16.7 #338

The ty bump to 0.0.80 surfaces one new diagnostic: asyncio.iscoroutinefunction is deprecated (removed in Python 3.16). Fixed by switching to inspect.iscoroutinefunction in src/sap_cloud_sdk/adms/_async_http.py. uv.lock is regenerated to match the widened ranges.

Related Issue

N/A

Type of Change

  • Dependency update
  • Bug fix (non-breaking change that fixes an issue) — deprecated-API fix required by the ty bump

How to Test

  1. uv sync
  2. uv run pre-commit run --all-filesruff format, ruff check, and ty check all pass.
  3. uv run pytest tests/adms/unit/ — all pass (covers the _bearer_token async/sync detection path that was changed).

Expected result: green lint/typecheck and passing unit tests with the updated dependency ranges.

Checklist

  • I have read the Contributing Guidelines
  • I have verified that my changes solve the issue
  • All tests pass locally
  • I have verified that my code follows the Code Guidelines
  • My code does not contain sensitive information (credentials, tokens, etc.)
  • I have followed Conventional Commits for commit messages
  • I have added/updated automated tests to cover my changes
  • I have updated documentation (if applicable)
  • I have added type hints for all public APIs

Breaking Changes

None

Additional Notes

None

- minio: <7.2.17 -> <7.2.21 (#339)
- setuptools: <84 -> <85 (#310)
- django (optional): <5 -> <6 (#305/#312)
- ty: 0.0.64 -> 0.0.80, ruff: 0.16.0 -> 0.16.7 (#338)

ty 0.0.80 flags the deprecated asyncio.iscoroutinefunction; switch
adms/_async_http.py to inspect.iscoroutinefunction. Regenerate uv.lock.

Branch cut for consumption by the a2a-agent requirements pin.
@jplbrun
jplbrun marked this pull request as ready for review September 16, 2026 19:42
@jplbrun
jplbrun requested a review from a team as a code owner September 16, 2026 19:42
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