Skip to content

ENG-3614/ENG-3823: Bump click and pydantic for litellm 1.84.0 compatibility#8178

Open
galvana wants to merge 6 commits into
mainfrom
ENG-3614-bump-click-aiohttp
Open

ENG-3614/ENG-3823: Bump click and pydantic for litellm 1.84.0 compatibility#8178
galvana wants to merge 6 commits into
mainfrom
ENG-3614-bump-click-aiohttp

Conversation

@galvana
Copy link
Copy Markdown
Contributor

@galvana galvana commented May 14, 2026

Summary

  • Bump click 8.1.7 → 8.1.8
  • Bump pydantic 2.12.4 → 2.12.5
  • Required for litellm 1.84.0 compatibility in fidesplus. Companion to ethyca/fidesplus#3574.

Test plan

  • CI passes (static checks, tests)
  • Verify no regressions in click-dependent code (CLI commands)
  • Verify no regressions in pydantic-dependent code (schemas, models)

🤖 Generated with Claude Code

Required for litellm >=1.83.7 compatibility (fidesplus CVE fixes).
Both are safe patch-level bumps with no breaking changes.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
fides-plus-nightly Ignored Ignored Preview May 16, 2026 3:20am
fides-privacy-center Ignored Ignored May 16, 2026 3:20am

Request Review

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 14, 2026

Dependency Review

✅ No vulnerabilities found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 6e67b06.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

  • uv.lock

litellm 1.83.14 pins aiohttp==3.13.4 (same as fides main) but
pydantic==2.12.5 (fides had 2.12.4). Revert unnecessary aiohttp
change and bump pydantic instead.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@galvana galvana changed the title ENG-3614: Bump click and aiohttp for litellm compatibility ENG-3614: Bump click and pydantic for litellm compatibility May 14, 2026
galvana and others added 2 commits May 15, 2026 20:15
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@galvana galvana changed the title ENG-3614: Bump click and pydantic for litellm compatibility ENG-3614/ENG-3823: Bump click and pydantic for litellm 1.84.0 compatibility May 16, 2026
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@codecov
Copy link
Copy Markdown

codecov Bot commented May 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.61%. Comparing base (7d5ba2c) to head (6e67b06).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8178   +/-   ##
=======================================
  Coverage   85.61%   85.61%           
=======================================
  Files         658      658           
  Lines       42869    42869           
  Branches     5019     5019           
=======================================
+ Hits        36701    36702    +1     
  Misses       5063     5063           
+ Partials     1105     1104    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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