Skip to content
Snippets Groups Projects

Cleanup codebase with `pyupgrade`

Merged Matt Harbison requested to merge topic/default/pyupgrade into branch/default

There might be some things that we don't want it to do (like drop the 'r' in open("foo", "r") just because it matches the default arg), but mostly this seems like a good thing, and has the ability to detect and re-write the legacy quoted type annotations that are sprinkled all over the codebase. We'll see what happens, but since this can fix the code in the file, maybe it would be a good candidate for an hg fix config and test-check-format.t before running black.

I left the questionable changes out of this, because it's already a lot of changes, and would like to land it before it bit rots. We can figure out the other changes after.

Edited by Matt Harbison

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading