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.

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

Support Pydantic v1 output #12

Merged
merged 3 commits into from
Nov 10, 2023
Merged

Support Pydantic v1 output #12

merged 3 commits into from
Nov 10, 2023

Conversation

lord-haffi
Copy link
Collaborator

@lord-haffi lord-haffi commented Nov 10, 2023

@DeltaDaniel might need this to work together with https://github.com/Hochfrequenz/bo4e-python-orm

@lord-haffi lord-haffi self-assigned this Nov 10, 2023
@@ -37,13 +37,21 @@ def generate_bo4e_schemas(input_directory: Path, output_directory: Path):
help="Output directory for the generated python files.",
required=True,
)
@click.option(
"--pydantic-v1/--pydantic-v2",
Copy link
Collaborator

Choose a reason for hiding this comment

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

wie gibt man das nun an?
kann man entweder --pydantic-v1 oder -pydantic-v2 angeben?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Genau. Und der default ist v2

@@ -37,13 +37,21 @@ def generate_bo4e_schemas(input_directory: Path, output_directory: Path):
help="Output directory for the generated python files.",
required=True,
)
@click.option(
"--pydantic-v1/--pydantic-v2",
"-p",
Copy link
Collaborator

Choose a reason for hiding this comment

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

und wie würde man in der abgekürzten Form angeben?
-p 1 vs -p 2 oder -p v1 ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@lord-haffi lord-haffi enabled auto-merge (squash) November 10, 2023 19:45
@lord-haffi lord-haffi merged commit f2ba5fa into main Nov 10, 2023
12 checks passed
@lord-haffi lord-haffi deleted the pydantic_v1_support branch November 10, 2023 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants