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.

Sign up for GitHub

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

Jump to bottom

Remote swagger validator has changed output format #1370

Closed
ml-evs opened this issue Oct 25, 2022 · 0 comments · Fixed by #1371
Closed

Remote swagger validator has changed output format #1370

ml-evs opened this issue Oct 25, 2022 · 0 comments · Fixed by #1371
Assignees
@ml-evs
Labels
bug Something isn't working CI Continuous Integration - GitHub Actions issues (NOT related to the repository Action) OpenAPI OpenAPI / Swagger related issue

Comments

@ml-evs
Copy link
Member

ml-evs commented Oct 25, 2022

The remote swagger validator at swagger.io has changed output format, which is breaking all of our CI tests.

It looks like it used to just return an empty JSON when validation was successful, but not contains an extra field for messages, so our invoke task will need to be updated.
@ml-evs ml-evs added bug Something isn't working OpenAPI OpenAPI / Swagger related issue CI Continuous Integration - GitHub Actions issues (NOT related to the repository Action) labels Oct 25, 2022
@ml-evs ml-evs self-assigned this Oct 25, 2022
@ml-evs ml-evs mentioned this issue Oct 25, 2022
Merged
@ml-evs ml-evs mentioned this issue Dec 6, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ml-evs ml-evs

Labels
bug Something isn't working CI Continuous Integration - GitHub Actions issues (NOT related to the repository Action) OpenAPI OpenAPI / Swagger related issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support new remote swagger.io validator format Materials-Consortia/optimade-python-tools
1 participant
@ml-evs