The Wayback Machine - https://web.archive.org/web/20201218090713/https://github.com/faker-ruby/faker/pull/1553
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

Yard powered docs #1553

Merged
merged 53 commits into from Sep 11, 2019
Merged

Yard powered docs #1553

merged 53 commits into from Sep 11, 2019

Conversation

@Zeragamba
Copy link
Contributor

@Zeragamba Zeragamba commented Feb 24, 2019

Closes #1330

@Zeragamba
Copy link
Contributor Author

@Zeragamba Zeragamba commented Feb 24, 2019

forgot to actually open this PR for #1330

@richardbulger
Copy link
Contributor

@richardbulger richardbulger commented Mar 8, 2019

Can CONTRIBUTING.md be updated with guidelines on how this should be used going forward?

@Zeragamba
Copy link
Contributor Author

@Zeragamba Zeragamba commented Apr 7, 2019

Updated contributing, and added some sample documentation to Faker::Alphanumeric, Faker::Book, and Faker::Books's subclasses.

@connorshea
Copy link
Contributor

@connorshea connorshea commented Jun 23, 2019

👍 this'd be great, especially for use with sord so it'd be easier to add Sorbet support (either officially in this repo or via sorbet-typed)

@vbrazo vbrazo force-pushed the faker-ruby:master branch 3 times, most recently from 69291b2 to 5cc660c Jul 5, 2019
@vbrazo
Copy link
Member

@vbrazo vbrazo commented Aug 7, 2019

@Zeragamba can you update this branch with master? We'll move on with these changes for v3.

@Zeragamba
Copy link
Contributor Author

@Zeragamba Zeragamba commented Aug 7, 2019

Copy link
Member

@bpleslie bpleslie left a comment

Looks good to me!

@Zeragamba Zeragamba force-pushed the Zeragamba:yard-powered-docs branch from be7ceb7 to b64fc6f Aug 12, 2019
lib/faker/creature/dog.rb Outdated Show resolved Hide resolved
@connorshea
Copy link
Contributor

@connorshea connorshea commented Aug 13, 2019

Also, you should probably add doc/ to the .gitignore since running yard causes a bunch of files to be added.

@connorshea
Copy link
Contributor

@connorshea connorshea commented Aug 13, 2019

For the @examples, you're formatting them like this:

# @example Faker::Game.genre
#   "Real-time strategy"

when normally it would be more like:

# @example 
#   Faker::Game.platform #=> "Nintendo Switch"

You can see the rendered documentation for these two here:

Screen Shot 2019-08-12 at 8 05 24 PM

I can make this change if you want it.

@stympy
Copy link
Collaborator

@stympy stympy commented Aug 13, 2019

I do prefer the latter style, @connorshea

@connorshea
Copy link
Contributor

@connorshea connorshea commented Aug 13, 2019

I think I've updated all of them now, that wasn't as hard as I expected. Zeragamba#6

@connorshea
Copy link
Contributor

@connorshea connorshea commented Aug 13, 2019

@vbrazo just to clarify, you’re not saying this PR would need to wait until 3.0, right?

We can have the current docs and the YARD docs simultaneously without any problem, and the current docs would be the main source until we’ve moved everything over to YARD.

@Zeragamba
Copy link
Contributor Author

@Zeragamba Zeragamba commented Aug 13, 2019

@connorshea Sounds good. Left a few change requests there, I'll merge those in when one of us get the time to fix it. :)

.gitignore Show resolved Hide resolved
@Zeragamba
Copy link
Contributor Author

@Zeragamba Zeragamba commented Aug 21, 2019

Lots of new docs from @connorshea so this is ready for re-review

@connorshea
Copy link
Contributor

@connorshea connorshea commented Aug 30, 2019

There are some new conflicts now.

@Zeragamba Zeragamba force-pushed the Zeragamba:yard-powered-docs branch 2 times, most recently from 100a035 to 8cfb1fc Sep 11, 2019
lib/faker/blockchain/bitcoin.rb Outdated Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
lib/faker/books/lovecraft.rb Outdated Show resolved Hide resolved
lib/faker/books/lovecraft.rb Outdated Show resolved Hide resolved
Co-Authored-By: Connor Shea <connor.james.shea@gmail.com>
Update CONTRIBUTING.md to match new example format
install Yard


and go go gadget yard


Revert "and go go gadget yard"

This reverts commit b2c8000.

fully document Faker::Blockchain


exclude deprecated methods


add version numbers to blockchain


update documentation to match what was discussed in #1330


remove class level @faker.version from docs

update CONTRIBUTING.md

update existing yard docs


clean up some of the docs
@Zeragamba Zeragamba force-pushed the Zeragamba:yard-powered-docs branch from 5edd8b7 to 2023e30 Sep 11, 2019
@Zeragamba
Copy link
Contributor Author

@Zeragamba Zeragamba commented Sep 11, 2019

Copy link
Contributor

@connorshea connorshea left a comment

LGTM 👍

@vbrazo
vbrazo approved these changes Sep 11, 2019
Copy link
Member

@vbrazo vbrazo left a comment

👍

@vbrazo vbrazo merged commit f9798d5 into faker-ruby:master Sep 11, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Zeragamba
Copy link
Contributor Author

@Zeragamba Zeragamba commented Sep 11, 2019

It merged! Woooo :party:

@connorshea
Copy link
Contributor

@connorshea connorshea commented Sep 11, 2019

🎉

vbrazo added a commit that referenced this pull request Sep 11, 2019
vbrazo added a commit that referenced this pull request Sep 11, 2019
vbrazo added a commit that referenced this pull request Sep 12, 2019
* Bump version to 2.3.0

* Add #1553 to 2.3.0

* Add connorshea to #1553

* Remove duplications

* Add #1741
@connorshea connorshea mentioned this pull request Sep 21, 2019
181 of 181 tasks complete
michebble added a commit to michebble/faker that referenced this pull request Feb 16, 2020
* install Yard

* add some basic docs to test with

* and go go gadget yard

* Revert "and go go gadget yard"

This reverts commit b2c8000.

* use square brackets not braces

* fully document Faker::Blockchain

* exclude deprecated methods

* add version numbers to blockchain

* update documentation to match what was discussed in faker-ruby#1330

* remove class level @faker.version from docs

* update CONTRIBUTING.md

* add documentation to Alphanumeric

* consistently use braces for types

* document the Books namespace

* add doc url

* Add yard docs for Faker::Blockchain::Tezos.block

* update docs for named params

* convert back to [] for types
Although YARD supports any kind of bracket for the Typing, square brackets are
used in the documentation. For those contributing, and unfamiliar with YARD, it
would be best that our documentation matches the same style as that in YARD's
tutorials and documentation.

* document Faker::Creature::Animal

* document Faker::Creature::Cat

* document Faker::Creature::Dog

* document Faker::Creature::Horse

* Update lib/faker/default/alphanumeric.rb

Co-Authored-By: Connor Shea <connor.james.shea@gmail.com>

* Update lib/faker/creature/dog.rb

Co-Authored-By: Connor Shea <connor.james.shea@gmail.com>

* document Faker::Game

* document Faker::Games::HalfLife

* document Faker::Games::Zelda

* document Faker::Games::SuperSmashBros

* document Faker::TvShows::BreakingBad

* Document Faker::Games::ElderScrolls

* document Faker::Gender

* document Faker::House

* document Faker::ProgrammingLanguage

* document Faker::Source

* Update some of the docs to use examples without titles.

* Update a few more docs to use examples without titles.

* Update more docs to use title-less examples.

* document Faker::Boolean

* document Faker::Avatar

* document Faker::Games::Fallout

* document Faker::Games::Overwatch

* document Faker::Hacker

* Fix a method description.

* Remove trailing whitespace.

* document Faker::App

* Add faker version for BreakingBad methods.

* Add faker version to Zelda methods.

* document Faker::Artist

* Merge the examples for semantic_version in app.rb.

* Add faker versions for elder_scrolls.rb and fallout.rb.

Some of the Elder Scrolls methods were added after others, in 1.9.0.

* Update version tags in Elder Scrolls and Fallout.

* Clean up docs

Co-Authored-By: Connor Shea <connor.james.shea@gmail.com>
Update CONTRIBUTING.md to match new example format
install Yard


and go go gadget yard


Revert "and go go gadget yard"

This reverts commit b2c8000.

fully document Faker::Blockchain


exclude deprecated methods


add version numbers to blockchain


update documentation to match what was discussed in faker-ruby#1330


remove class level @faker.version from docs

update CONTRIBUTING.md

update existing yard docs


clean up some of the docs

* fix line ending issues
michebble added a commit to michebble/faker that referenced this pull request Feb 16, 2020
* Bump version to 2.3.0

* Add faker-ruby#1553 to 2.3.0

* Add connorshea to faker-ruby#1553

* Remove duplications

* Add faker-ruby#1741
@Zeragamba Zeragamba deleted the Zeragamba:yard-powered-docs branch May 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.