Skip to content

Update docs regarding the args yielded to blocks#108

Merged
philipqnguyen merged 1 commit intomasterfrom
fix-block-docs
Oct 18, 2018
Merged

Update docs regarding the args yielded to blocks#108
philipqnguyen merged 1 commit intomasterfrom
fix-block-docs

Conversation

@philipqnguyen
Copy link
Copy Markdown
Contributor

The blocks are yielded both the object and the options that were passed
to render. This commit simply ensure that the docs reflects that.

@mcclayton
Copy link
Copy Markdown
Contributor

Very nice 👍

Comment thread lib/blueprinter/base.rb Outdated
# to the :default view.
# @yield [Object] The object passed to `render` is also passed to the
# block.
# @yield [object, options] The object and the options passed to render is
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Oh, perhaps are instead of is.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ty

Comment thread lib/blueprinter/base.rb Outdated
# The method, proc or string should return or evaluate to a true or false value.
# @yield [Object] The object passed to `render` is also passed to the
# block.
# @yield [object, options] The object and the options passed to render is
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Oh, perhaps are instead of is.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ty

The blocks are yielded both the object and the options that were passed
to render. This commit simply ensure that the docs reflects that.
@philipqnguyen philipqnguyen merged commit f849049 into master Oct 18, 2018
@jmeridth jmeridth deleted the fix-block-docs branch July 26, 2023 15:31
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.

2 participants