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

Fix DeprecationWarnings #1417

Merged
merged 4 commits into from
Feb 1, 2019
Merged

Fix DeprecationWarnings #1417

merged 4 commits into from
Feb 1, 2019

Conversation

astrojuanlu
Copy link
Contributor

No description provided.

@astrojuanlu
Copy link
Contributor Author

Oops, sorry for the rush, didn't realize that collections.abc does not exist in Legacy Python (aka Python 2). Is there a Python 2 roadmap?

@jonmmease
Copy link
Contributor

Thanks a lot @Juanlu001. We don't have a definitely plan on Python 2 yet except that we don't plan to drop support in the near tern. Would it be enough to add some try catch handling to the import?

@jonmmease
Copy link
Contributor

Add exception handling and going to merge. Thanks @Juanlu001!

@jonmmease jonmmease merged commit 7978caf into plotly:master Feb 1, 2019
@jonmmease jonmmease added this to the v3.5.0 milestone Feb 1, 2019
@astrojuanlu
Copy link
Contributor Author

Sorry it took me a bit long to respond to this one, busy week. Thanks for completing it!

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.

None yet

2 participants