Skip to content
This repository has been archived by the owner on Sep 11, 2019. It is now read-only.

fix: allow gramps() calls with no options #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: allow gramps() calls with no options #86

wants to merge 1 commit into from

Conversation

jlengstorf
Copy link
Member

  • Remove rest/spread for gramps() args
  • Add single options object with a default

close #81, close #84

@jlengstorf
fix: allow gramps() calls with no options
e01fc59 
- Remove rest/spread for `gramps()` args
- Add single options object with a default

close #81, close #84
@jlengstorf
Copy link
Member Author

@ryanomackey If you've got a second to try this out and verify that it actually solves the problem in #81, I'd appreciate it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ecwyne ecwyne Awaiting requested review from ecwyne

@ryanomackey ryanomackey Awaiting requested review from ryanomackey

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.

Calling gramps() with no arguments causes an error
1 participant
@jlengstorf