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 destination a dict instead a tuple #48

Closed
jinpark opened this issue May 22, 2015 · 5 comments
Closed

Make destination a dict instead a tuple #48

jinpark opened this issue May 22, 2015 · 5 comments

Comments

@jinpark
Copy link

jinpark commented May 22, 2015

Since there are a lot of options now, would it be easier to make it into a dict, instead of a tuple?

@jinpark
Copy link
Author

jinpark commented May 22, 2015

I can totally take this btw, just wanted to make sure that you're ok with it :)

@bradleyg
Copy link
Owner

Sounds good, I've been wondering if there is a clean way to do it without breaking API... but I don't think there is.

@jinpark
Copy link
Author

jinpark commented May 27, 2015

Probably a good way is to create a new version and notify that it breaks backwards compatibility.

@bradleyg
Copy link
Owner

ok, sounds like a fair compromise.

@ivanvenosdel
Copy link
Contributor

Just saw this. Excellent idea. I think a way this can be done in a backwards compatible manner is to do some duck-typing to determine if the old version is in use and then internally convert the current style into the same style of dict as the forward version. Those using the old style will be given a deprecation warning and, after the PyPI release, the docs would be updated to show the new style. Eventually, at some TBD date support for the old version could be dropped.

If someone doesn't beat me to it, I will make a note to see if I can fit this in with #62. I think the feature needed for #62 can require the new version as a way to encourage adoption. The dict I spoke of in that ticket can be an 'extra_options' key in the new style.

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

No branches or pull requests

3 participants