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

Add YARD doc for Faker::Code #1961

Closed
wants to merge 1 commit into from

Conversation

cubeguerrero
Copy link
Contributor

Issue#

#1762

Description:

The PR adds YARD docs for Faker::Code

# @exmaple
# Faker::Code.npi #=> "9804062802"
#
# @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.

I believe the faker version should be the first where the method was released and available to usage

Copy link
Contributor

@Zeragamba Zeragamba left a comment

Choose a reason for hiding this comment

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

other then wrong @faker.versions, docs look good

# 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.

4 participants