Skip to content
This repository has been archived by the owner on Mar 11, 2019. It is now read-only.

Change documentation to make it straight forward #95 #108

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

musale
Copy link
Contributor

@musale musale commented Aug 15, 2017

Re-structure the documentation so that it flows from use case, setup and usage.

Description
The current documentation lacks a straightforward flow and presentation. This attempts to address the issue making it easier to navigate the documentation.

Reference to official issue
Covers raised issue #95. Even though much hasn't been explained in the issue, this PR attempts to address the basic documentation issues.

Motivation and Context
This will improve user documentation for the library

How Has This Been Tested?
Locally after running docker-compose up, navigate to http://localhost:7000/

re-structure the documentation so that it flows from use case, setup
and usage.
@codecov
Copy link

codecov bot commented Aug 15, 2017

Codecov Report

Merging #108 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #108   +/-   ##
=======================================
  Coverage   91.84%   91.84%           
=======================================
  Files          34       34           
  Lines        1116     1116           
=======================================
  Hits         1025     1025           
  Misses         91       91
Impacted Files Coverage Δ
ussd/core.py 97.13% <100%> (ø) ⬆️

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 6bb6928...e3db2dc. Read the comment docs.

@mwaaas
Copy link
Owner

mwaaas commented Sep 17, 2017

Hey @musale thanks for this pull request.
Would please resolve conflicts

@musale
Copy link
Contributor Author

musale commented Sep 18, 2017

Hi @mwaaas! I have done the merge

Repository owner deleted a comment Sep 18, 2017
Repository owner deleted a comment Sep 18, 2017
Copy link
Owner

@mwaaas mwaaas left a comment

Choose a reason for hiding this comment

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

My comment:
+1 LGTM

Copy link
Owner

@mwaaas mwaaas left a comment

Choose a reason for hiding this comment

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

My comment:
+1 testing

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants