Opened 2 years ago

Closed 2 years ago

#29773 closed enhancement (fixed)

Remove a few unnessary future imports

Reported by: gh-tobiasdiez Owned by:
Priority: minor Milestone: sage-9.2
Component: misc Keywords:
Cc: Merged in:
Authors: Tobias Diez Reviewers: Matthias Koeppe
Report Upstream: N/A Work issues:
Branch: 8d84c43 (Commits, GitHub, GitLab) Commit: 8d84c436878cb61c19db470c23711d1c6804de37
Dependencies: Stopgaps:

Status badges

Description

This PR removes a few future imports. These are obsolete with Python 3.7. See https://docs.python.org/3/reference/simple_stmts.html#future-statements. There are quite a few more of these future imports, which I can remove if you would like.

Moreover, sometimes an error message occurred because the DeprecationWarning wasn't an element of the warnings.filters list. Thus, I added an check for existence before removing it.

Change History (4)

comment:1 Changed 2 years ago by gh-tobiasdiez

  • Status changed from new to needs_review

comment:2 Changed 2 years ago by mkoeppe

  • Authors set to Tobias Diez

comment:3 Changed 2 years ago by mkoeppe

  • Reviewers set to Matthias Koeppe
  • Status changed from needs_review to positive_review

comment:4 Changed 2 years ago by vbraun

  • Branch changed from u/gh-tobiasdiez/removeFuture to 8d84c436878cb61c19db470c23711d1c6804de37
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.