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

Backfill initial yard docs #52

Merged
merged 1 commit into from Oct 6, 2020
Merged

Backfill initial yard docs #52

merged 1 commit into from Oct 6, 2020

Conversation

ms-ati
Copy link
Contributor

@ms-ati ms-ati commented Oct 5, 2020

First pass through the MemoWise module. Includes all of the non-metaprogrammed class and instance methods. Excludes all of the meta-programmed methods, which will come in a separate PR after we figure out the best way to document them.

We add examples in the YARD doc comments which are tested using the
yard doctest plugin.

lib/memo_wise.rb Outdated Show resolved Hide resolved
lib/memo_wise.rb Outdated Show resolved Hide resolved
lib/memo_wise.rb Show resolved Hide resolved
lib/memo_wise.rb Show resolved Hide resolved
@ms-ati
Copy link
Contributor Author

ms-ati commented Oct 5, 2020

See #34

@ms-ati ms-ati force-pushed the backfill-initial-yard-docs branch 4 times, most recently from 7473fa3 to 7aa734d Compare October 6, 2020 14:41
lib/memo_wise.rb Show resolved Hide resolved
lib/memo_wise.rb Show resolved Hide resolved
* Add yard-doctest and run in CI on Ruby 2.7
* Add --no-private yardopt to hide private APIs from docs
* Doc MemoWise.has_arg? (and make private)
* Doc MemoWise.has_kwarg? (and make private)
* Doc MemoWise.prepended as private, and
* Doc MemoWise#initialize
* Doc MemoWise#reset_memo_wise
* Doc MemoWise#preset_memo_wise
* Remove reset_all_memo_wise (add to reset_memo_wise)
* Fix GFM table (benchmark results) - add redcarpet dependency
* Fix benchmarks to run on new Ruby patch versions, e.g. 2.7.2
@ms-ati ms-ati merged commit c83d315 into main Oct 6, 2020
@ms-ati ms-ati deleted the backfill-initial-yard-docs branch October 6, 2020 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants