Skip to content

Conversation

@sydney-runkle
Copy link
Contributor

@sydney-runkle sydney-runkle commented Nov 22, 2024

Fixes #10946
Fixes #9491

This might not be the best approach, but relieves some pressure for now. Leans into the coupling between pydantic and pydantic-core. I think it's better to lean into this than to pretend it doesn't exist.

Working on docs test fixes

@github-actions github-actions bot added the relnotes-fix Used for bugfixes. label Nov 22, 2024
@codspeed-hq
Copy link

codspeed-hq bot commented Nov 22, 2024

CodSpeed Performance Report

Merging #10947 will not alter performance

Comparing easier-ser (a84e306) with main (15f693a)

Summary

✅ 46 untouched benchmarks

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Nov 22, 2024

Deploying pydantic-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: a84e306
Status: ✅  Deploy successful!
Preview URL: https://9dc69f81.pydantic-docs.pages.dev
Branch Preview URL: https://easier-ser.pydantic-docs.pages.dev

View logs

@github-actions
Copy link
Contributor

github-actions bot commented Nov 22, 2024

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  pydantic/_internal
  _fields.py
  _generate_schema.py
  _typing_extra.py
Project Total  

This report was generated by python-coverage-comment-action

Copy link
Contributor

@davidhewitt davidhewitt left a comment

Choose a reason for hiding this comment

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

This seems fine for now 👍

It would be nice in the long run to either decide this is what we want users to do too, or to find a better way to support types with __get_pydantic_core_schema__.

@sydney-runkle sydney-runkle enabled auto-merge (squash) November 25, 2024 15:14
@sydney-runkle sydney-runkle merged commit 1667c84 into main Nov 25, 2024
52 checks passed
@sydney-runkle sydney-runkle deleted the easier-ser branch November 25, 2024 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

relnotes-fix Used for bugfixes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2.10: Trouble serializing url types (as any), similar to secret types Problem when serializing models with SecretStrs

4 participants