Skip to content

Add support for exclude_if at the field level #12141

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add support for exclude_if at the field level #12141

wants to merge 2 commits into from

Conversation

Viicos
Copy link
Member

@Viicos Viicos commented Aug 7, 2025

Change Summary

Supersedes #10961.

Related issue number

Checklist

  • The pull request title is a good summary of the changes - it will be used in the changelog
  • Unit tests for the changes exist
  • Tests pass on CI
  • Documentation reflects the changes where applicable
  • My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

@github-actions github-actions bot added the relnotes-fix Used for bugfixes. label Aug 7, 2025
@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Aug 7, 2025 •
edited
Loading

Deploying pydantic-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7d41d11
Status: ✅  Deploy successful!
Preview URL: https://318dcf9a.pydantic-docs.pages.dev
Branch Preview URL: https://exclude-if.pydantic-docs.pages.dev

View logs

@Viicos Viicos mentioned this pull request Aug 7, 2025
Closed
5 tasks
tonnico
tonnico reviewed Aug 7, 2025
View reviewed changes
@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Aug 7, 2025 •
edited
Loading

CodSpeed Performance Report

Merging #12141 will not alter performance

Comparing exclude-if (7d41d11) with main (1c79f0e)

Summary

✅ 46 untouched benchmarks

@Viicos @andresliszt
Add support for exclude_if at the field level
eac6893 
Co-authored-by: Andres <andres.sandoval@alumnos.usm.cl>
@Viicos Viicos added relnotes-feature needs-blogpost-entry This PR needs to be documented in the release notes blog post and removed relnotes-fix Used for bugfixes. labels Aug 7, 2025
@Viicos Viicos requested a review from DouweM August 7, 2025 10:44
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 7, 2025

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)		Lines missing
  pydantic
  fields.py
Project Total  

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tonnico tonnico tonnico left review comments

@DouweM DouweM Awaiting requested review from DouweM

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
needs-blogpost-entry This PR needs to be documented in the release notes blog post relnotes-feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Viicos @tonnico