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

Auto-generate API documentation. Refactor code. #40

Merged
merged 1 commit into from
Sep 18, 2014
Merged

Auto-generate API documentation. Refactor code. #40

merged 1 commit into from
Sep 18, 2014

Conversation

Ichimonji10
Copy link
Contributor

Generate API documentation directly from docstrings. Re-write several docstrings
to be clearer and use Sphinx directives.

Refactor the logic behind codify and FauxFactory.generate_string.

@@ -10,9 +10,7 @@ Getting Started
sys.path.append(ROOT_PATH)
from fauxfactory import FauxFactory

FauxFactory generates random data for your automated tests easily!
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Already stated in introduction.

@Ichimonji10
Copy link
Contributor Author

Not all docstrings have been reworked. Only the docstrings for three (?) functions and methods have been reworked.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) when pulling 4c005b3 on Ichimonji10:api-docs into 3927cd9 on omaciel:master.

Generate API documentation directly from docstrings. Re-write several docstrings
to be clearer and use Sphinx directives.

Refactor the logic behind `codify` and `FauxFactory.generate_string`.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.02%) when pulling d54047b on Ichimonji10:api-docs into 3927cd9 on omaciel:master.

else:
raise Exception("%s is not a supported string type." % str_type)
method = getattr(cls, 'generate_{}'.format(str_type_lower))
Copy link
Owner

Choose a reason for hiding this comment

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

Niice

@omaciel
Copy link
Owner

omaciel commented Sep 18, 2014

ACK

omaciel added a commit that referenced this pull request Sep 18, 2014
Auto-generate API documentation. Refactor code.
@omaciel omaciel merged commit 46a722b into omaciel:master Sep 18, 2014
@Ichimonji10 Ichimonji10 deleted the api-docs branch September 22, 2014 14:05
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.

None yet

3 participants