Skip to content

Py2k Decimation: first pass, removing compatibility.py (#1127)#99

Merged
sthagen merged 1 commit intosthagen:developfrom
streamlit:develop
Mar 1, 2020
Merged

Py2k Decimation: first pass, removing compatibility.py (#1127)#99
sthagen merged 1 commit intosthagen:developfrom
streamlit:develop

Conversation

@sthagen
Copy link
Copy Markdown
Owner

@sthagen sthagen commented Mar 1, 2020

  • removes 2_3 shims

  • replace six.string_types with str

  • reducing some warnings in the tests

  • resolve deprecation warnings

  • fix mypy python3.8 complaint

Before contributing (PLEASE READ!)

⚠️ As with most projects, prior to starting to code on a bug fix or feature request, please post in the issue saying you want to volunteer, and then wait for a positive response. And if there is no issue for it yet, create it first.

This helps make sure (1) two people aren't working on the same thing, (2) this is something Streamlit's maintainers believe should be implemented/fixed, (3) any API, UI, or deeper architectural changes that need to be implemented have been fully thought through by Streamlit's maintainers.

More information in our wiki: https://github.com/streamlit/streamlit/wiki/Contributing


Issue: Please include a link to the issue you're addressing. If no issue exists, create one first and then link it here.

Description: Describe the changes you made to the code, so it's easier for the reader to navigate your pull request. Usually this is a bullet list.


Contribution License Agreement

By submiting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

* removes 2_3 shims 

* replace six.string_types with str

* reducing some warnings in the tests

* resolve deprecation warnings

* fix mypy python3.8 complaint
@sthagen sthagen merged commit 88effb1 into sthagen:develop Mar 1, 2020
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