-
Notifications
You must be signed in to change notification settings - Fork 75
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
Removed invalid descriptions on root definitions in rtp.json #65
base: main
Are you sure you want to change the base?
Removed invalid descriptions on root definitions in rtp.json #65
Conversation
Thank you for your pull request and welcome to our community! To contribute, please sign the Oracle Contributor Agreement (OCA). To sign the OCA, please create an Oracle account and sign the OCA in Oracle's Contributor Agreement Application. When signing the OCA, please provide your GitHub username. After signing the OCA and getting an OCA approval from Oracle, this PR will be automatically updated. If you are an Oracle employee, please make sure that you are a member of the main Oracle GitHub organization, and your membership in this organization is public. |
@rb80 Please review this for merge into your existing PR to main |
Thank you for signing the OCA. |
@alexander-applyinnovations , I think this is resolved with #64 which among other things includes the removal of unicode from specs. If you agree could you close this PR? |
@rb80 could you please consider merging this PR or please specify why this can't be done? Thanks! |
The current definition of the updated docs contains two lines that do not satisfy the validation requirements for open-api spec.
This PR amends the rtp api spec to remove the invalid description fields in order to generate sdks using the openapi sdk generator.
Here are the relevant error logs:
GitHub actions log for
applyinnovations/ohip-sdk
respositoryhttps://github.com/applyinnovations/ohip-sdk/actions/runs/6634049598/job/18022861256