Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork2.2k
Add acheck_pydantic_core_version()
function#11324
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.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Ensuring the correct `pydantic-core` version is installed.
cloudflare-workers-and-pagesbot commentedJan 23, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Deploying pydantic-docs with |
Latest commit: | 6d632ea |
Status: | ✅ Deploy successful! |
Preview URL: | https://6859aff4.pydantic-docs.pages.dev |
Branch Preview URL: | https://check-pc-version.pydantic-docs.pages.dev |
codspeed-hqbot commentedJan 23, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
CodSpeed Performance ReportMerging#11324 willnot alter performanceComparing Summary
|
github-actionsbot commentedJan 23, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Coverage reportClick to see where and how coverage changed
This report was generated bypython-coverage-comment-action |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
We need a comment on pyproject.toml saying "you need to update this in xxx if you change it here."
Uh oh!
There was an error while loading.Please reload this page.
@@ -63,6 +65,12 @@ def version_info() -> str: | |||
return '\n'.join('{:>30} {}'.format(k + ':', str(v).replace('\n', ' ')) for k, v in info.items()) | |||
def check_pydantic_core_version() -> bool: | |||
"""Check that the installed `pydantic-core` dependency is compatible.""" | |||
# Keep this in sync with the version constraint in the `pyproject.toml` dependencies: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Can we automate this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Probably overkill imo, considering the added test fails if this function was not updated after apydantic-core
version bump
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Sounds good, let's merge.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGTM, though I'd prefer an automated update...
ea3f18d
intomainUh oh!
There was an error while loading.Please reload this page.
Ensuring the correct
pydantic-core
version is installed.Closes#11323.
Change Summary
Related issue number
Checklist