OpenInterpreter / 01

The open-source language model computer
http://openinterpreter.com/01
GNU Affero General Public License v3.0
4.76k stars 470 forks source link

Fix pre-commit linter #209

Closed dheavy closed 3 months ago

dheavy commented 3 months ago

Closes #208.

Problem

Ruff linting not linting what should be linted (software directory).

Solution

Add actual software directory to the list in the pre-commit ruff config.

Tested on

Discussion

This PR, albeit necessary for smoother collaboration, generates a lot of diff in many files—the actual linting. Therefore, it has been broken into two distinct commits. This will facilitate review (only the first commit is actually proceeding with fixing the pre-commit issue) and should help any regression investigation via git bisect, if ever needed.

Tests from the Tested on section also include actually running the CLI simulator to ensure there were no regression.