-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Complete YARD docs #1762
Comments
Hey @connorshea |
@ashishra0 it should be updated now, I don't check things off until they've been merged into master. |
@connorshea Yeah makes sense. Thanks though! |
That's pretty awesome 💥 |
I am still in chapter 3 of my course, I need someone to put me through... Better on Twitter @Melebroz1 |
I've updated the issue description with all the new PRs :) |
The last one's left is Faker::Code which has an open PR already. |
@danielTiringer thank you for all the work! 🙏 Looks like @Zeragamba has created the updated PR for Faker::Code, so that'll be the last one we need to merge and then this'll be finished :) |
And that's the last one merged to master! |
Thanks for the great work everybody! |
That was awesome! Thanks! |
Here are all the Faker classes that have been updated with YARD docs/need to be updated:
Add YARD docs for Faker:Verb #1916Add YARD docs to Faker::Verb #2008)The text was updated successfully, but these errors were encountered: