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

Add Faker::Markdown.sandwich #1064

Merged
merged 8 commits into from
May 28, 2018

Conversation

russellschmidt
Copy link
Contributor

Another take on issue 1052... this creates a fake markdown text block starting with a header and then a variable number of sentences in a lorem ipsum paragraph and then a random markdown element. Users can repeat that duo of lorem paragraph and random markdown element if they want longer text blocks.

Since I am recycling other methods, I wasn't totally sure what to test for here other than the iteration on a default case had the right number of pieces.

Copy link
Member

@vbrazo vbrazo left a comment

Choose a reason for hiding this comment

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

Could you rebase with master and fix the Rubocop violations?

Could you also add your name + PR to the changelog.md?

@vbrazo vbrazo changed the title Markdown sandwich Add Faker::Markdown.sandwich May 28, 2018
Copy link
Member

@vbrazo vbrazo left a comment

Choose a reason for hiding this comment

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

Thanks for contributing 🥇

Overall the PR was ok. I had free time today and pushed a few small changes. Thanks again.

@vbrazo vbrazo merged commit 26048ab into faker-ruby:master May 28, 2018
@krsnachandra
Copy link

When will it be released? I'd love to use this!

@vbrazo
Copy link
Member

vbrazo commented Jun 2, 2018

You could point to our master branch and use it until we release a new version. Only the owner could release the 1.8.8, so I'm sure when it'll happen.

@vbrazo
Copy link
Member

vbrazo commented Jul 19, 2018

@krsnachandra we released a new version this month.

@vbrazo vbrazo self-requested a review July 19, 2018 01:24
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* corrected typo on dune.md

* get rid of ahead of myself dune.md changes

* add documentation and extended sandwich artistry

* tests pass ready to party

* remove dune.md changes on my machine

* Update changelog

* Add missing sandwich tests
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