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 yard docs for Faker::Coffee methods #1810

Merged
merged 2 commits into from Nov 11, 2019

Conversation

LuanGB
Copy link
Contributor

@LuanGB LuanGB commented Oct 15, 2019

Add yard docs for Faker::Coffee methods

Copy link
Member

@connorshea connorshea left a comment

Choose a reason for hiding this comment

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

Other than the versions, looks good to me

# @example
# Faker::Coffee.blend_name #=> "Major Java"
#
# @faker.version next
Copy link
Member

Choose a reason for hiding this comment

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

The methods for Faker::Coffee were added in 1.8.0, except for intensifier which was added in 1.9.0. All of these should be changed to reflect that.

Copy link
Member

@bpleslie bpleslie left a comment

Choose a reason for hiding this comment

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

Looks good other than @connorshea comment. Thanks for the contribution!

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.

👍

@vbrazo vbrazo merged commit fdf95cf into faker-ruby:master Nov 11, 2019
@vbrazo
Copy link
Member

vbrazo commented Nov 11, 2019

@LuanGB welcome!

@connorshea connorshea mentioned this pull request Nov 11, 2019
sanikkenway added a commit to sanikkenway/faker that referenced this pull request Nov 12, 2019
Add yard docs for Faker::Coffee methods (faker-ruby#1810)
michebble pushed a commit to michebble/faker that referenced this pull request Feb 16, 2020
* Add yard docs for Faker::Coffee methods

* @faker.version should be 1.9.0
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* Add yard docs for Faker::Coffee methods

* @faker.version should be 1.9.0
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.

None yet

4 participants