open-interpreter icon indicating copy to clipboard operation
open-interpreter copied to clipboard

Changed pre-commit formatter from Black to Ruff

Open SuibhneOFoighil opened this issue 1 year ago • 3 comments

Describe the changes you have made:

Added Ruff, changed pre-commit script and relevant docs (contributing.md)

Reference any relevant issues (e.g. "Fixes #000"):

Fixes "Change formatter from Black to Ruff #849"

Pre-Submission Checklist (optional but appreciated):

  • [ x] I have included relevant documentation updates (stored in /docs)
  • [ x] I have read docs/CONTRIBUTING.md
  • [ ] I have read docs/ROADMAP.md

OS Tests (optional but appreciated):

  • [x] Tested on Windows
  • [x] Tested on MacOS
  • [ ] Tested on Linux

SuibhneOFoighil avatar Mar 28 '24 11:03 SuibhneOFoighil

LMK if I can do anything to improve it!

SuibhneOFoighil avatar Mar 29 '24 23:03 SuibhneOFoighil

@SuibhneOFoighil, I'll test on Windows for you! Edit: it works on Windows, you can tick that box too!

dheavy avatar Mar 31 '24 20:03 dheavy

Hi @SuibhneOFoighil, thanks for tackling this and thanks @Notnaton and @dheavy for testing! As Anton said I think it would be smart to move it to dev dependencies, then will merge. Great work!

KillianLucas avatar Apr 17 '24 01:04 KillianLucas