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

Fix Java OpenAPI generation for REST API (0.39) #2451

Merged
merged 1 commit into from
Aug 25, 2021

Conversation

longnguyen1997
Copy link
Contributor

  • Fix 400 ref bug in OpenAPI spec
  • Add properly named response objects for major API calls and delete additional_properties on state proofs
  • Add Java generation of OpenAPI spec for validation

Signed-off-by: Long Nguyen [email protected]

Description:

Related issue(s):

Fixes #

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

* Fix 400 ref bug in OpenAPI spec
* Add properly named response objects for major API calls and delete additional_properties on state proofs
* Add Java generation of OpenAPI spec for validation

Signed-off-by: Long Nguyen <[email protected]>
@codecov
Copy link

codecov bot commented Aug 25, 2021

Codecov Report

Merging #2451 (1433313) into release/0.39 (35d4ecd) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##             release/0.39    #2451   +/-   ##
===============================================
  Coverage           90.83%   90.83%           
+ Complexity           2433     2432    -1     
===============================================
  Files                 420      420           
  Lines               11593    11593           
  Branches             1013     1013           
===============================================
  Hits                10530    10530           
  Misses                732      732           
  Partials              331      331           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 35d4ecd...1433313. Read the comment docs.

@steven-sheehy steven-sheehy added this to the Mirror 0.39.0 milestone Aug 25, 2021
@steven-sheehy steven-sheehy requested a review from a team August 25, 2021 19:52
@steven-sheehy steven-sheehy added bug Type: Something isn't working P2 rest Area: REST API labels Aug 25, 2021
@steven-sheehy steven-sheehy changed the title Fix Java OpenAPI generation for REST API (#2447) Fix Java OpenAPI generation for REST API (0.39) Aug 25, 2021
Copy link
Member

@steven-sheehy steven-sheehy left a comment

Choose a reason for hiding this comment

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

LGTM

@longnguyencircle
Copy link
Contributor

longnguyencircle commented Aug 25, 2021

Mirror gRPC / Redis is failing or something? Is this related to anything I did?

@steven-sheehy
Copy link
Member

I think it's just a transient error. Retrying.

Copy link
Collaborator

@xin-hedera xin-hedera left a comment

Choose a reason for hiding this comment

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

LGTM

@steven-sheehy
Copy link
Member

Error looks to be due to a regression in k3s 1.21.3 that is fixed in 1.21.4: k3s-io/k3s#3704. We can merge this and I'll pin the k3s version in another PR.

@longnguyencircle
Copy link
Contributor

Thank you!

@steven-sheehy steven-sheehy merged commit 632b698 into hashgraph:release/0.39 Aug 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Type: Something isn't working P2 rest Area: REST API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants