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

Use default warning type for the get_raw_runs_for_flow_id deprecation #112

Merged
merged 2 commits into from
Oct 1, 2020

Conversation

as2388
Copy link
Member

@as2388 as2388 commented Sep 29, 2020

Review #111 first.

DeprecationWarnings don't show by default in 3.6, and I think it's more effort than it's worth to set a filter on all the projects. Worse, the filter defaults change across minor versions of Python, and matplotlib (and possibly other libraries) bypasses the built-in filters anyway by creating its own MatplotlibDeprecationWarning, so I think this is the simplest "solution"

… warning instead of DeprecationWarning

DeprecationWarnings don't show by default in 3.6, and I think it's more effort than it's worth to set a filter on all the projects. Worse, the filter defaults change across minor versions of Python, and matplotlib (and possibly other libraries) bypasses the built-in filters anyway by creating its own MatplotlibDeprecationWarning.
@as2388 as2388 requested a review from IsaackMwenda September 29, 2020 17:04
@as2388 as2388 merged commit 9ac2c0b into master Oct 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