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

Investigate and fix deprecation warnings encountered in tests #3988

Closed
gshank opened this issue Oct 1, 2021 · 0 comments · Fixed by #4556
Closed

Investigate and fix deprecation warnings encountered in tests #3988

gshank opened this issue Oct 1, 2021 · 0 comments · Fixed by #4556
Assignees
Labels
tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality

Comments

@gshank
Copy link
Contributor

gshank commented Oct 1, 2021

When we run our tests we get a fairly large number of deprecation warnings. Many of these seem to come from dependencies. We should investigate these warnings and see if we should 1) upgrade the dependency or 2) modify our code or 2) just wait.

Here are some examples of warnings. We should run the tests and collect all of them for investigation.

env/lib/python3.8/site-packages/packaging/version.py:127: DeprecationWarning: Creating a LegacyVersion has been deprecated and will be removed in the next major release

jinja2/runtime.py:679: DeprecationWarning: 'soft_unicode' has been renamed to 'soft_str'. The old name will be removed in MarkupSafe 2.1.

<class 'networkx.utils.decorators.argmap'> compilation 5:5: DeprecationWarning: write_gpickle is deprecated and will be removed in 3.0.Use pickle.dump(G, path, protocol)

@jtcohen6 jtcohen6 added the tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality label Nov 18, 2021
@gshank gshank self-assigned this Jan 4, 2022
gshank added a commit that referenced this issue Jan 7, 2022
iknox-fa pushed a commit that referenced this issue Feb 8, 2022
automatic commit by git-black, original commits:
  5aeb088
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tech_debt Behind-the-scenes changes, with little direct impact on end-user functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants