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

update v10 manifest #55

Merged
merged 1 commit into from
Aug 17, 2023
Merged

update v10 manifest #55

merged 1 commit into from
Aug 17, 2023

Conversation

dlawin
Copy link
Contributor

@dlawin dlawin commented Aug 16, 2023

Update manifest to version hosted here https://schemas.getdbt.com/dbt/manifest/v10.json

Updated generated classes:

(.venv) ➜  dev git:(main) ✗ datamodel-codegen  --input-file-type jsonschema \
    --disable-timestamp \
    --base-class "dbt_artifacts_parser.parsers.base.BaseParserModel" \
    --class-name "ManifestV10" \
    --input "../dbt_artifacts_parser/resources/manifest/manifest_v10.json" \
    --output "../dbt_artifacts_parser/parsers/manifest/manifest_v10.py"

This is to resolve #54

@yu-iskw
Copy link
Owner

yu-iskw commented Aug 17, 2023

@dlawin Thank you for the contribution!

@yu-iskw yu-iskw merged commit 1660752 into yu-iskw:main Aug 17, 2023
5 checks passed
@yu-iskw
Copy link
Owner

yu-iskw commented Aug 17, 2023

@dlawin 0.4.2 was out. Can you give it a try?

https://github.com/yu-iskw/dbt-artifacts-parser/releases/tag/v0.4.2

@dlawin
Copy link
Contributor Author

dlawin commented Aug 17, 2023

@dlawin 0.4.2 was out. Can you give it a try?

https://github.com/yu-iskw/dbt-artifacts-parser/releases/tag/v0.4.2

yep I'll give it a try later today!

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.

ManifestV10 parsing errors on full 1.6 release
2 participants