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

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants