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 open api spec linting errors #8

Closed
wants to merge 2 commits into from

Conversation

Jongmassey
Copy link
Contributor

I linted the icar-ade-openapispec yaml using the speccy linter (http://speccy.io/) which found a non-resolving url reference (see #7) and that the milking-visit-id parameter of many methods was missing a description. Description for this parameter was populated from icarMilkingVisitResource.json

@cookeac
Copy link
Collaborator

cookeac commented Jun 19, 2019

Erwin's changes to the YAML file have removed these items back to the JSON schema files (now included by reference) and we have checked that these descriptions are in place in those files.

@cookeac cookeac closed this Jun 19, 2019
erwinspeybroeck pushed a commit that referenced this pull request Aug 19, 2019
erwinspeybroeck pushed a commit that referenced this pull request Apr 22, 2020
Merge from adewg/ICAR
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.

2 participants