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

Fix deprecation warnings, fixes #177 #178

Closed

Conversation

mmaslowskicc
Copy link
Collaborator

No description provided.

@coveralls
Copy link

coveralls commented Aug 16, 2016

Coverage Status

Coverage increased (+0.05%) to 94.676% when pulling de68f1b on mmaslowskicc:deprecation-warnings into 6784172 on ClearcodeHQ:master.

@mmaslowskicc mmaslowskicc force-pushed the deprecation-warnings branch 2 times, most recently from aae4f61 to 17fe1cf Compare August 16, 2016 11:55
@coveralls
Copy link

coveralls commented Aug 16, 2016

Coverage Status

Coverage increased (+0.02%) to 94.646% when pulling 17fe1cf on mmaslowskicc:deprecation-warnings into 6784172 on ClearcodeHQ:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 94.737% when pulling 17fe1cf on mmaslowskicc:deprecation-warnings into 6784172 on ClearcodeHQ:master.

@coveralls
Copy link

coveralls commented Aug 16, 2016

Coverage Status

Coverage increased (+0.07%) to 94.701% when pulling 2179da1 on mmaslowskicc:deprecation-warnings into 6784172 on ClearcodeHQ:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 94.646% when pulling 7b957ed on mmaslowskicc:deprecation-warnings into 6784172 on ClearcodeHQ:master.

@coveralls
Copy link

coveralls commented Aug 16, 2016

Coverage Status

Coverage remained the same at 94.628% when pulling 7b957ed on mmaslowskicc:deprecation-warnings into 6784172 on ClearcodeHQ:master.

@mmaslowskicc
Copy link
Collaborator Author

Or should I fix the current warnings without checking them during CI builds? (This finds them in several dependencies on Python 3.5, and future changes might cause them to raise more.)

@fizyk
Copy link
Member

fizyk commented Aug 17, 2016

@mmaslowskicc either that, or turn specific packages warnings into errors, that occur in our code.

@fizyk fizyk closed this Dec 7, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants