Revert isort call in pre-commit to ensure consistent errors with CI linter

[trey0]

The isort pre-commit call was edited here so that it calls isort only on changed files rather than recursively on the whole repo.

But there was a comment just above that line explaining in absurd detail why we are calling it recursively on the whole repo...

[trey0]

Note that if the motivation for the change was to exclude submodules from linting, it looks like that can be done with the --extend-skip flag as used in the corresponding call in the astrobee repo.

[trey0]

Forgot to note this was fixed as part of 7d7852b