Skip to content
This repository has been archived by the owner on Jan 19, 2022. It is now read-only.

allow for nested opts parameter in options #6

Merged
merged 1 commit into from
Dec 8, 2020
Merged

allow for nested opts parameter in options #6

merged 1 commit into from
Dec 8, 2020

Conversation

nlf
Copy link
Contributor

@nlf nlf commented Dec 8, 2020

this is because the cli sends these options nested under a second opts parameter, so we need to also flatten that into our options here to allow for things like --searchopts=from=2

References

@nlf nlf mentioned this pull request Dec 8, 2020
Merged
@nlf @isaacs
allow for nested opts parameter in options
78cdf81 
PR-URL: #6
Credit: @nlf
Close: #6
Reviewed-by: @isaacs
@isaacs isaacs force-pushed the nlf/extra-options branch from 483ec61 to 78cdf81 Compare December 8, 2020 23:51
@isaacs isaacs closed this in 78cdf81 Dec 8, 2020
@isaacs isaacs merged commit 78cdf81 into latest Dec 8, 2020
@wraithgar wraithgar deleted the nlf/extra-options branch September 21, 2021 16:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@darcyclarke darcyclarke darcyclarke approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nlf @darcyclarke @isaacs