rope icon indicating copy to clipboard operation
rope copied to clipboard

Better Tooling

Open bagel897 opened this issue 2 years ago • 2 comments

Could we use and configure the following tools in rope?

  1. tox - testing across python versions. Would need to test from 3.6/7 to 3.10/3.11
  2. isort - sorting the imports. Would need to configure to use the black profile. Would need to reformat almost all the files
  3. pre-commit - use black and isort before commits to prevent us from having to make additional blacken commits.

If approved, I can add these, either as one PR or 3 seperate PRs. The most problematic is isort since it would reformat every file in rope.

bagel897 avatar Jun 05 '22 16:06 bagel897

+1 on all these tools.

If there's a mass reformatting, they should be their own separate commit and PR.

lieryan avatar Jun 17 '22 12:06 lieryan

So I just did this for a project that I work on. Super big quality of life improvement. You should just be able to copy a lot of our settings. We use black, isort, and precommit. Also, I have a lot of it explained on our contributor page.

project

contributing docs

Rowlando13 avatar Sep 18 '22 21:09 Rowlando13