ion icon indicating copy to clipboard operation
ion copied to clipboard

Clean up code

Open JasonGrace2282 opened this issue 1 year ago • 2 comments
trafficstars

Most of these were automatically fixed by running pre-commit (#1659 ).

Note that most of these should be pretty safe, as ruff was run without the --unsafe-fixes flag, and ruffhas been designed around not modifying the resulting runtime code so most of it should run the same

99% of changes fall into one of the 3 categories;

  • Typos
  • "...".format(...) to f"..."
  • End of line endings to unix end of line character

EDIT: I removed changes from all vendor files to reduce the diff (it went from a 3,000 character diff to a 500 character diff). If the maintainers of Ion want vendor files to be fixed, I can make another PR for that.

JasonGrace2282 avatar May 03 '24 20:05 JasonGrace2282

Coverage Status

coverage: 79.417% (+0.006%) from 79.411% when pulling 5837a9f15fbc46f162d23f350c34481e34f0cb27 on JasonGrace2282:precommit-lint-project into 3fd1236933dbf64650ef6576466fc3dd3b5c4c15 on tjcsl:dev.

coveralls avatar May 20 '24 03:05 coveralls