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

Adds a table of contents to the readme file. #613

Merged
merged 1 commit into from
Jun 23, 2015
Merged

Adds a table of contents to the readme file. #613

merged 1 commit into from
Jun 23, 2015

Conversation

camilopayan
Copy link
Contributor

I found it really hard to get around the readme to answer my questions quickly. I wasn't familiar enough with it to search for what I needed. To that end, I created a readme file to go with it.

Let me know if there are any changes you want me to make. Great library, btw, I'm really enjoying using it.

- [Installation](#installation)
- [Basic Usage](#basic-usage)
- [Formatters](#formatters)
- [`FakerProviderBase`](#fakerproviderbase)
Copy link
Owner

Choose a reason for hiding this comment

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

No need to repeat FakerProvider, or to make them code formatted. I'd simply label subsections Base, Lorem, Person, etc.

@fzaninotto
Copy link
Owner

Great idea!

I found it really hard to get around the readme to answer my questions
quickly. I wasn't familiar enough with it to search for what I needed.
To that end, I created a readme file to go with it.
@camilopayan
Copy link
Contributor Author

All right, updated the pull request per your notes. I agree with the notes, btw, wasn't sure if you'd want it code formatted, figured it'd be best to just pass it by you first.

fzaninotto added a commit that referenced this pull request Jun 23, 2015
Adds a table of contents to the readme file.
@fzaninotto fzaninotto merged commit 3d45dcf into fzaninotto:master Jun 23, 2015
@fzaninotto
Copy link
Owner

Awesome, thanks a lot!

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