Skip to content

Remove schema validation #1723

New issue

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

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

Already on GitHub? # to your account

Draft
wants to merge 10 commits into
base: remove-schema-validation
Choose a base branch
from

Conversation

clin1234
Copy link

Change Summary

Rebase upstream's remove-schema-validation branch with python-3.14 branch, alongside minor linting

Complements #1714

Related issue number

fix #1714

Checklist

  • Unit tests for the changes exist
  • Documentation reflects the changes where applicable
  • Pydantic tests pass with this pydantic-core (except for expected changes)
  • My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Copy link

codecov bot commented May 27, 2025

Codecov Report

Attention: Patch coverage is 10.00000% with 9 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/validators/dataclass.rs 0.00% 3 Missing ⚠️
src/validators/model.rs 0.00% 2 Missing ⚠️
src/serializers/filter.rs 0.00% 1 Missing ⚠️
src/tools.rs 50.00% 1 Missing ⚠️
src/validators/enum_.rs 0.00% 1 Missing ⚠️
src/validators/model_fields.rs 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Viicos
Copy link
Member

Viicos commented May 27, 2025

@clin1234 what is the purpose of this PR? We will handle the 3.14 support through the two already open PRs.

@clin1234
Copy link
Author

@clin1234 what is the purpose of this PR? We will handle the 3.14 support through the two already open PRs.

For 3.14 support to be added, both of the aformentioned PRs would have to be merged; this is just an sandbox to test when one of the PRs is rebased on top of another (mainly CI tests)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants