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

docs: Update airbyte use case + fix broken link #2021

Merged
merged 2 commits into from
Jul 1, 2024

Conversation

natikgadzhi
Copy link
Contributor

What

This pull request fixes a broken link to Airbyte's use case, and clarifies how we use datamodel-code-generators as well.

How

README.md change.

User Impact

None, should be a safe merge.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Copy link

codecov bot commented Jul 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (d0778c5) to head (a58c281).

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #2021   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           38        38           
  Lines         4188      4188           
  Branches       961       961           
=========================================
  Hits          4188      4188           
Flag Coverage Δ
unittests 99.66% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

codspeed-hq bot commented Jul 1, 2024

CodSpeed Performance Report

Merging #2021 will not alter performance

Comparing natikgadzhi:patch-1 (a58c281) with main (d0778c5)

Summary

✅ 29 untouched benchmarks

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@koxudaxi koxudaxi merged commit 9ac0425 into koxudaxi:main Jul 1, 2024
92 checks passed
bpsoos pushed a commit to bpsoos/datamodel-code-generator that referenced this pull request Jul 1, 2024

Partially verified

This commit is signed with the committer’s verified signature.
bpsoos’s contribution has been verified via SSH key.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
Co-authored-by: Koudai Aono <koxudaxi@gmail.com>
koxudaxi added a commit that referenced this pull request Jul 4, 2024

Partially verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
We cannot verify signatures from co-authors, and some of the co-authors attributed to this commit require their commits to be signed.
…r --use-annotated (#2016)

* 🩹 fixes graphql parser use-standard-collections and use-union-operator

* 🩹 fixes graphql parser to use annoteted on typename fields

* ♻️ refactors tests

* ♻️ refactors tests

Signed-off-by: bpsoos <soos.peter.96@gmail.com>

* ♻️ refactors tests

Signed-off-by: bpsoos <soos.peter.96@gmail.com>

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* 🩹 fixes failing test

Signed-off-by: bpsoos <soos.peter.96@gmail.com>

* ♻️ refactors test_main into multiple files

Signed-off-by: bpsoos <soos.peter.96@gmail.com>

* ♻️ refactors test_main into multiple files

Signed-off-by: bpsoos <soos.peter.96@gmail.com>

* 🩹 fixes msgspec output file name

Signed-off-by: bpsoos <soos.peter.96@gmail.com>

* ♻️ refactors jsonschema expectations

Signed-off-by: bpsoos <soos.peter.96@gmail.com>

* ♻️ refactors jsonschema expectations

Signed-off-by: bpsoos <soos.peter.96@gmail.com>

* ♻️ refactors jsonschema expectations

Signed-off-by: bpsoos <soos.peter.96@gmail.com>

* ♻️ refactors openapi expectations

Signed-off-by: bpsoos <soos.peter.96@gmail.com>

* ♻️ refactors csv json yaml and general expectations

Signed-off-by: bpsoos <soos.peter.96@gmail.com>

* Fix test coverage (#2020)

* Fix test coverage

* Fix test coverage

* Update output.py

* Fix test coverage

* Fix coverage

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Fix coverage

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Fix coverage

* Fix coverage

* Fix coverage

* Fix coverage

* Fix coverage

* Fix coverage

* Fix coverage

* Fix coverage

* Fix coverage

* Fix coverage

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* Fix coverage (#2023)

* Fix coverage

* Fix coverage

* docs: Update airbyte use case + fix broken link (#2021)

Co-authored-by: Koudai Aono <koxudaxi@gmail.com>

* formats

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Signed-off-by: bpsoos <soos.peter.96@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Koudai Aono <koxudaxi@gmail.com>
Co-authored-by: Natik Gadzhi <natik@respawn.io>
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.

None yet

2 participants