Skip to content
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

linkml 1.6.4 eliminates old pydantic_version cli parameter #1780

Closed
sierra-moxon opened this issue Dec 12, 2023 · 1 comment · May be fixed by #1781
Closed

linkml 1.6.4 eliminates old pydantic_version cli parameter #1780

sierra-moxon opened this issue Dec 12, 2023 · 1 comment · May be fixed by #1781
Labels
bug Something that should work but isn't, with an example and a test case. generator-pydantic

Comments

@sierra-moxon
Copy link
Member

Describe the bug
Models that use 1.6.3 or earlier run a command like this to generate pydantic serialization of the model:
$(RUN) gen-pydantic --pydantic_version 1 src/model/schema/model.yaml > $(PYMODEL)/pydanticmodel.py

In 1.6.4 they should run:
$(RUN) gen-pydantic --pydantic-version 1 src/model/schema/model.yaml > $(PYMODEL)/pydanticmodel.py

we should support the underscore version for a few releases to help people transition.
@sierra-moxon sierra-moxon added the bug Something that should work but isn't, with an example and a test case. label Dec 12, 2023
@sierra-moxon sierra-moxon mentioned this issue Dec 12, 2023
Open
@caufieldjh caufieldjh mentioned this issue Dec 12, 2023
Closed
@sierra-moxon
Copy link
Member Author

closing as complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something that should work but isn't, with an example and a test case. generator-pydantic
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test cli for pydantic gen and add deprecated support for --pydantic_v… linkml/linkml
2 participants
@nlharris @sierra-moxon