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 missing description field to operation params #276

Merged
merged 2 commits into from May 2, 2019

Conversation

Projects
None yet
2 participants
@andresanches
Copy link
Contributor

commented May 2, 2019

Parameter descriptions provide good insight on specificities of what values are expected for each parameter. The field is optional and is described as such

Contributing to Twilio

All third party contributors acknowledge that any contributions they provide will be made under the same open source license that the open source project is provided under.

  • I acknowledge that all my contributions will be made under the project's license.
Add missing description field to operation params
Parameter descriptions provide good insight on specificities of what values are expected for each parameter
@blast-hardcheese
Copy link
Collaborator

left a comment

Not yet used in guardrail, but offering description as scaladoc/javadoc comments would be very beneficial. Having this here is a reminder.

Thanks!

@blast-hardcheese blast-hardcheese merged commit 54ce238 into twilio:master May 2, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.