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 documentation #28

Merged
merged 7 commits into from
Jul 24, 2017
Merged

Add api documentation #28

merged 7 commits into from
Jul 24, 2017

Conversation

jan-xyz
Copy link
Contributor

@jan-xyz jan-xyz commented Jul 18, 2017

This fixes issue #22.

I added the documentation found in my previous pull request to the documentation of the tool. I also added 3 sentences about the API usage. Let me know if that is all it needs or if you want more!

Jan

Jan Steinke added 5 commits July 18, 2017 12:56
 This is a direct copy of the text from the pull request. It still needs formatting and probably a bit more context.
 * move mentioning of how to download images up
 * fix a typo
 * correct code blop in image download section
@codecov-io
Copy link

codecov-io commented Jul 18, 2017

Codecov Report

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

@@         Coverage Diff          @@
##           master   #28   +/-   ##
====================================
  Coverage      90%   90%           
====================================
  Files          12    12           
  Lines         500   500           
====================================
  Hits          450   450           
  Misses         50    50

@rhsimplex
Copy link
Contributor

Made a PR to your PR =)

If the changes are ok, accept and I'll merge it.

Thanks!

@jan-xyz
Copy link
Contributor Author

jan-xyz commented Jul 24, 2017

Thanks for the Feedback! Highly appreciated!

@rhsimplex rhsimplex merged commit 4046b53 into merantix:master Jul 24, 2017
@jan-xyz jan-xyz deleted the add-api-documentation branch July 25, 2017 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants