Description review edits For Redis Cache swaggers#673
Description review edits For Redis Cache swaggers#673amarzavery merged 3 commits intoAzure:masterfrom
Conversation
|
Hi @steved0x, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
TTYL, AZPRBOT; |
|
Most of the checklist above doesn't apply to me since I did not author the swagger - I just edited the descriptions on a previously approved and merged swagger. |
|
#sign-off |
| "info": { | ||
| "title": "RedisManagementClient", | ||
| "description": ".Net client wrapper for the REST API for Azure Redis Cache Management Service", | ||
| "description": ".Net client wrapper for the REST API for Azure Redis Cache Management Service.", |
There was a problem hiding this comment.
@steved0x - Dont you think the description should not have keywords like .NET and client in it? The spec is a representation of the API and would be used to generate clients in different languages. Plus it wouldnt look good to have the word client specified in the ref docs.
|
@amarzavery I made the requested changes, thank you. |
|
No modification for Python |
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information
api-versionin the path should match theapi-versionin the spec).Quality of Swagger
This change is