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

forgot to actually open this PR for #1330

@richardbulger
Copy link
Contributor

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

@Zeragamba
Copy link
Contributor Author

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

@connorshea
Copy link
Member

👍 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 master branch 3 times, most recently from 69291b2 to 5cc660c Compare July 5, 2019 17:21
@vbrazo
Copy link
Member

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 commented Aug 7, 2019 via email

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 to me!

@connorshea
Copy link
Member

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

@connorshea
Copy link
Member

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
Contributor

stympy commented Aug 13, 2019

I do prefer the latter style, @connorshea

@connorshea
Copy link
Member

I think I've updated all of them now, that wasn't as hard as I expected. https://github.com/Zeragamba/faker/pull/6

@connorshea
Copy link
Member

@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 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

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

@connorshea
Copy link
Member

There are some new conflicts now.

@Zeragamba Zeragamba force-pushed the yard-powered-docs branch 2 times, most recently from 100a035 to 8cfb1fc Compare September 11, 2019 01:39
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
Copy link
Contributor Author

live example of these docs: https://rubydoc.info/github/SpyMaster356/faker/master/Faker/Avatar

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.

LGTM 👍

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 f9798d5 into faker-ruby:master Sep 11, 2019
@Zeragamba
Copy link
Contributor Author

It merged! Woooo :party:

@connorshea
Copy link
Member

🎉

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
michebble pushed 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 pushed 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 yard-powered-docs branch May 17, 2020 00:21
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* 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
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
* 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
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.

Possibility of moving to YARD-style documentation in README
6 participants