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

H-454: Update Python OpenAPI generation to include filters #2932

Merged
merged 5 commits into from
Aug 18, 2023

Conversation

indietyp
Copy link
Member

🌟 What is the purpose of this PR?

This updates the OpenAPI generation to include filters upon generation, something I forgot in the previous PR (#2868)

Pre-Merge Checklist 🚀

🚢 Has this modified a publishable library?

This PR:

  • does not modify any publishable blocks or libraries, or modifications do not need publishing

📜 Does this require a change to the docs?

The changes in this PR:

  • are internal and do not require a docs change

🕸️ Does this require a change to the Turbo Graph?

The changes in this PR:

  • do not affect the execution graph

@github-actions github-actions bot added area/apps > hash* Affects HASH (a `hash-*` app) area/infra Relates to version control, CI, CD or IaC (area) area/libs Relates to first-party libraries/crates/packages (area) type/eng > backend Owned by the @backend team area/apps area/apps > hash-graph labels Aug 18, 2023
@indietyp indietyp requested a review from TimDiekmann August 18, 2023 08:03
apps/hash/README.md Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Aug 18, 2023

Codecov Report

Merging #2932 (e7cf182) into main (f172803) will decrease coverage by 0.28%.
Report is 4 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2932      +/-   ##
==========================================
- Coverage   51.93%   51.65%   -0.28%     
==========================================
  Files         337      345       +8     
  Lines       29399    29644     +245     
  Branches      437      476      +39     
==========================================
+ Hits        15269    15314      +45     
- Misses      14128    14324     +196     
- Partials        2        6       +4     
Flag Coverage Δ
hash-graph 54.82% <ø> (ø)

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

see 8 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions github-actions bot removed the area/infra Relates to version control, CI, CD or IaC (area) label Aug 18, 2023
@indietyp indietyp enabled auto-merge August 18, 2023 08:39
@indietyp indietyp added this pull request to the merge queue Aug 18, 2023
Merged via the queue into main with commit bffb448 Aug 18, 2023
@indietyp indietyp deleted the bm/graph-sdk/py-codegen branch August 18, 2023 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/apps > hash* Affects HASH (a `hash-*` app) area/apps > hash-graph area/apps area/libs Relates to first-party libraries/crates/packages (area) type/eng > backend Owned by the @backend team
Development

Successfully merging this pull request may close these issues.

3 participants