Skip to content
This repository has been archived by the owner on Jun 13, 2018. It is now read-only.

Replace options with keyword args #430

Open
kmcphillips opened this issue Dec 3, 2016 · 1 comment
Open

Replace options with keyword args #430

kmcphillips opened this issue Dec 3, 2016 · 1 comment

Comments

@kmcphillips
Copy link
Member

Since we'll be dropping lower version support, we can use keyword args everywhere. #426

So many of the method signatures are like (a, b, c, d, e, options={}) and those should be replaced with documented keyword arguments.

@kmcphillips kmcphillips added this to the Version 2.0 milestone Dec 3, 2016
@kmcphillips
Copy link
Member Author

In any case where we still use option, in the case that we do, we need to symbolize_keys! because we are inconsistent about that now.

@garethson garethson removed this from the Version 2.0 milestone May 30, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants