Skip to content

ref(pydantic-ai): Use hooks.on.model_request to replace individual hooks - #7327

Open
alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/pydantic/wrap-model-request
Open

ref(pydantic-ai): Use hooks.on.model_request to replace individual hooks#7327
alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/pydantic/wrap-model-request

Conversation

@alexander-alderman-webb

Copy link
Copy Markdown
Contributor

Description

Noticed that we can simplify while working on the major.

Use hooks.on.model_request to replace all of hooks.on.before_model_request, hooks.on.model_request_error and hooks.on.after_model_request.

Issues

Reminders

@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner September 1, 2026 12:49
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

121930 passed | ⏭️ 6762 skipped | Total: 128692 | Pass Rate: 94.75% | Execution Time: 415m 5s

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests 📈 +1
Failed Tests 📉 -1
Skipped Tests

➖ Removed Tests (1)

View removed tests
  • test_continuous_profiler_auto_start_and_stop_sampled[experiment-thread]
    • File: tests.profiler.test_continuous_profiler

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2484 uncovered lines.
✅ Project coverage is 90.29%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    90.27%    90.29%    +0.02%
==========================================
  Files          193       193         —
  Lines        25649     25580       -69
  Branches      9454      9418       -36
==========================================
+ Hits         23153     23096       -57
- Misses        2496      2484       -12
- Partials      1445      1428       -17

Generated by Codecov Action

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