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

set default options #63

Merged
merged 1 commit into from
Jul 8, 2020
Merged

set default options #63

merged 1 commit into from
Jul 8, 2020

Conversation

tsargent
Copy link
Contributor

@tsargent tsargent commented Jul 7, 2020

Setting default options might make things a tiny bit more readable, and removes a few lines of undefined checks. Not an enormous gain, but thought I would take a stab at a contribution 😄

Thanks for looking!

@yury-dymov
Copy link
Owner

yay! That's a great contribution, thank you!

@yury-dymov yury-dymov merged commit 55154ab into yury-dymov:master Jul 8, 2020
@tsargent tsargent deleted the set-default-options branch July 9, 2020 00:58
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