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

Add api:validate:local command and refactor shared functions #306

Merged
merged 5 commits into from
Jul 3, 2023

Conversation

CalemRoelofsSB
Copy link
Contributor

@CalemRoelofsSB CalemRoelofsSB commented Jun 29, 2023

This PR:

  • Adds a new api:validate:local command that takes an API definition from the local filesystem and runs a standardization scan on it using the given organization's standardization configuration on SwaggerHub.
  • Adds support for json output to the api:validate/api:validate:local commands as requested by Missing features #110 (comment)
  • Updates the output from the api:validate/api:validate:local commands to use ux.table to bring it in line with the tables in the rest of the codebase
  • I have added the appropriate label to my PR

@CalemRoelofsSB CalemRoelofsSB added the enhancement New feature or request label Jun 29, 2023
README.md Outdated Show resolved Hide resolved
`

ValidateLocalCommand.examples = [
'swaggerhub api:validate-local -c -j -o myOrg -f ./my-api.yaml',
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above - instead of :

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for catching that one, sorted now

@CalemRoelofsSB CalemRoelofsSB merged commit fd5b801 into main Jul 3, 2023
@CalemRoelofsSB CalemRoelofsSB deleted the feature/add-validate-local-command branch July 3, 2023 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants