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

docs: wrapped can take empty string as name #213

Closed
wants to merge 1 commit into from

Conversation

wpyoga
Copy link
Contributor

@wpyoga wpyoga commented Sep 2, 2022

As discussed in #212, wrapped can take an empty string as the name argument. In this case, the parsed object is directly embedded into the current object, similar to calling nest without the optional name parameter.

@keichi
Copy link
Owner

keichi commented Sep 4, 2022

@wpyoga Can I close this PR? #215 seems to include the changes in this PR.

@wpyoga
Copy link
Contributor Author

wpyoga commented Sep 4, 2022

@keichi If you intend to merge both updates, then yes you can close this one. I split these up so that you may merge either one independently.

@keichi
Copy link
Owner

keichi commented Sep 5, 2022

Closing in favor of #215

@keichi keichi closed this Sep 5, 2022
@wpyoga wpyoga deleted the doc-updates branch September 5, 2022 16:30
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