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.

Sign up for GitHub

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

Jump to bottom

Remove Warnings from tests #1133

Open

cbeauchesne opened this issue Jul 23, 2021 · 1 comment

Labels

chore

Member

cbeauchesne commented Jul 23, 2021 •

edited

Loading

Let's close this issue when warning count will be 0 on tests.

2020-07-22: 1932
2020-07-23: 0 (but 1842 deprecation warnings)

cbeauchesne added the chore label

cbeauchesne mentioned this issue

Warning count from 1932 to 1842 #1134

Merged

Member Author

cbeauchesne commented Jul 23, 2021

Status on 2020-07-23:

Most of warning are DeprecationWarning. And the pain part is that, often, they come from 3rd party lib. For each of them we must check if :

we need to update our code
or we need to update the lib

So, for now, I've ignored them, because I think there is no sense to bother devs for day-to-day dev. And I'll try to reduce the count time to time.

And by the way, there is no more warning, except Deprecation warning 😎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment