-
Notifications
You must be signed in to change notification settings - Fork 307
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
Fixing references to OpenAPI #355
base: main
Are you sure you want to change the base?
Conversation
Now should align with the guidelines in the OpenAPI style guide reference here: https://github.com/OAI/OpenAPI-Style-Guide
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the updates! |
Yes, but these 'fixes' are only for the first main page - what about all the .md files? If you feel passionate about this, could please fix these as well? Thanks! |
1. **Compile:** from a Swagger JSON or YAML specification (and optionally examples), generate a RESTler grammar. See [Compiling](./docs/user-guide/Compiling.md). | ||
2. **Test:** execute quickly all of the endpoints+methods in a compiled RESTler grammar for debugging the test setup and compute what parts of the Swagger spec are covered. This mode is also called a *smoketest*. | ||
1. **Compile:** from an OpenAPI document in either JSON or YAML format (and optionally examples), generate a RESTler grammar. See [Compiling](./docs/user-guide/Compiling.md). | ||
2. **Test:** execute quickly all of the endpoints+methods in a compiled RESTler grammar for debugging the test setup and compute what parts of the OpenAPI Specification are covered. This mode is also called a *smoketest*. | ||
See [Testing](./docs/user-guide/Testing.md). To use custom test engine settings, see [Test Engine Settings](./docs/user-guide/SettingsFile.md). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there should not be an upper case to 'Specification' here.
@@ -66,8 +66,8 @@ Prerequisites: Install [Python 3.8.2](https://www.python.org/downloads/) and | |||
|
|||
RESTler runs in 4 main modes (in order): | |||
|
|||
1. **Compile:** from a Swagger JSON or YAML specification (and optionally examples), generate a RESTler grammar. See [Compiling](./docs/user-guide/Compiling.md). | |||
2. **Test:** execute quickly all of the endpoints+methods in a compiled RESTler grammar for debugging the test setup and compute what parts of the Swagger spec are covered. This mode is also called a *smoketest*. | |||
1. **Compile:** from an OpenAPI document in either JSON or YAML format (and optionally examples), generate a RESTler grammar. See [Compiling](./docs/user-guide/Compiling.md). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please replace 'document' by 'specification' for consistency
RESTler intelligently infers producer-consumer dependencies among request types from the Swagger | ||
specification. During testing, it checks for specific classes of bugs | ||
RESTler intelligently infers producer-consumer dependencies among request types from the OpenAPI | ||
document. During testing, it checks for specific classes of bugs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please replace 'document' by 'specification' for consistency
@@ -91,7 +91,7 @@ When a bug is found, RESTler reports bugs triaged in bug buckets, and provides a | |||
|
|||
## Advanced Topics | |||
|
|||
For tips on using RESTler effectively, please see [Best Practices](./docs/user-guide/BestPractices.md) and [Improving Swagger Coverage](./docs/user-guide/ImprovingCoverage.md). | |||
For tips on using RESTler effectively, please see [Best Practices](./docs/user-guide/BestPractices.md) and [Improving OpenAPI Coverage](./docs/user-guide/ImprovingCoverage.md). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'OpenAPI' should really be 'API' here.
PS: feel free to merge and I can clean this up later. Thanks! |
Now should align with the guidelines in the OpenAPI style guide reference here: https://github.com/OAI/OpenAPI-Style-Guide