Skip to content

Conversation

@dpopp07
Copy link
Contributor

@dpopp07 dpopp07 commented May 4, 2018

I put some docs into the README for IAM. I basically just followed the format @lpatino10 used in the Java SDK. Let me know what y'all think, happy to change things around.

README.md Outdated
* [IBM Watson Services](#ibm-watson-services)
* [Authorization](#authorization)
* [Assistant]
* [Assistant] (#assistant)
Copy link
Contributor

Choose a reason for hiding this comment

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

The space in between the [] and () needs to be removed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch

@codecov-io
Copy link

Codecov Report

Merging #689 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #689   +/-   ##
=======================================
  Coverage   85.15%   85.15%           
=======================================
  Files          35       35           
  Lines        4115     4115           
  Branches      520      520           
=======================================
  Hits         3504     3504           
  Misses        255      255           
  Partials      356      356

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ec4a652...26d9996. Read the comment docs.

@anweshan anweshan merged commit 1c25604 into master May 4, 2018
@dpopp07 dpopp07 deleted the add-iam-docs branch May 7, 2018 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants