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

chore: upgrade to openapi generator 7.11 #2517

Merged
merged 7 commits into from
Jan 27, 2025

Conversation

edgarvonk
Copy link
Member

Upgraded to openapi generator version 7.11. One of our issues was fixed in this version but we got two new issues back for which I added workarounds.. It looks like both are caused by the new support for JSON-B polymorphism type annotations introduced by OpenAPITools/openapi-generator#20164:

Solves PZ-5160

@edgarvonk edgarvonk requested review from hsiliev and a team as code owners January 24, 2025 15:52
Copy link
Contributor

unit-test-results

 79 files  ±0  109 suites  ±0   2m 4s ⏱️ -6s
378 tests ±0  377 ✅ ±0  1 💤 ±0  0 ❌ ±0 
472 runs  ±0  470 ✅ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit eb3a0c3. ± Comparison against base commit 891cd86.

Copy link
Contributor

integration-test-results

 28 files  ±0   28 suites  ±0   35s ⏱️ ±0s
107 tests ±0  107 ✅ ±0  0 💤 ±0  0 ❌ ±0 
120 runs  ±0  120 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit eb3a0c3. ± Comparison against base commit 891cd86.

Copy link

codecov bot commented Jan 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.62%. Comparing base (916165b) to head (eb3a0c3).
Report is 19 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##               main    #2517    +/-   ##
==========================================
  Coverage     54.62%   54.62%            
  Complexity     3360     3360            
==========================================
  Files          1000     1000            
  Lines         22746    22746            
  Branches       1750     1777    +27     
==========================================
  Hits          12426    12426            
+ Misses         9525     9406   -119     
- Partials        795      914   +119     
Flag Coverage Δ
integrationtests 50.00% <100.00%> (ø)
unittests 34.69% <100.00%> (ø)

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.

@xiduzo xiduzo added this pull request to the merge queue Jan 27, 2025
Merged via the queue into main with commit c277176 Jan 27, 2025
23 checks passed
@xiduzo xiduzo deleted the feature/PZ-5160-upgrade-to-openapi-generator-7-11 branch January 27, 2025 10:46
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.

3 participants