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

Make name parameter optional for wrapped #215

Merged
merged 4 commits into from
Sep 5, 2022

Conversation

wpyoga
Copy link
Contributor

@wpyoga wpyoga commented Sep 2, 2022

This is just like nest, where if the name parameter is omitted, then the parsed object is embedded directly into the current object.

The update for the readme assumes that #213 has been merged.

@lgtm-com
Copy link

lgtm-com bot commented Sep 2, 2022

This pull request fixes 1 alert when merging 1a5daad into 8eacd3e - view on LGTM.com

fixed alerts:

  • 1 for Useless conditional

@keichi keichi merged commit b047862 into keichi:master Sep 5, 2022
@keichi
Copy link
Owner

keichi commented Sep 5, 2022

FYI I just published a new release on npm.

@wpyoga wpyoga deleted the wrapped-optional-name 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