Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added documentation for dumb and dumber #1135

Merged
merged 3 commits into from
May 17, 2018
Merged

Added documentation for dumb and dumber #1135

merged 3 commits into from
May 17, 2018

Conversation

cnharris10
Copy link
Contributor

@cnharris10 cnharris10 commented Jan 27, 2018

Adding Dumb and Dumber codex to README.

"I expected the Rocky Mountains README.md to be a little rockier more detailed than this."
-Harry Dunn

“I was thinking the same thing. That John Denver’s Chris Harris's full of shit, man.”
-Lloyd Christmas

@@ -0,0 +1,11 @@
# Faker::Seinfeld
Copy link
Member

Choose a reason for hiding this comment

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

Copy and 🍝

@vbrazo
Copy link
Member

vbrazo commented May 14, 2018

@cnharris10 Looks good 👍

Just two minor fixes. Please write a great description for this PR as well.

README.md Outdated
@@ -53,6 +53,7 @@ Contents
- [Faker::DragonBall](doc/dragon_ball.md)
- [Faker::DrWho](doc/dr_who.md)
- [Faker::Dune](doc/dune.md)
- [Faker::DumbAndDumber](doc/dumb_and_dumber.md)
Copy link
Member

@vbrazo vbrazo May 14, 2018

Choose a reason for hiding this comment

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

It should be ordered by name, so DumbAndDumber should come first.

@cnharris10
Copy link
Contributor Author

cnharris10 commented May 16, 2018

alt text

Thanks

@coveralls
Copy link

Pull Request Test Coverage Report for Build 2027

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 33.923%

Totals Coverage Status
Change from base Build 2022: 0.0%
Covered Lines: 1476
Relevant Lines: 4351

💛 - Coveralls

@faker-ruby faker-ruby deleted a comment from coveralls May 16, 2018
@faker-ruby faker-ruby deleted a comment from coveralls May 16, 2018
@faker-ruby faker-ruby deleted a comment from coveralls May 16, 2018
@vbrazo vbrazo merged commit c43298d into faker-ruby:master May 17, 2018
@vbrazo vbrazo self-requested a review July 19, 2018 01:40
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* Added documentation for dumb and dumber

* Update README.md

* Update dumb_and_dumber.md
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants