This reverts commit a7c5d887e943aa51f2270e517954c024a8c01500.

`flake8` seems to dislike the formatting of black.

That way people who use it won't be deterred, while it unifies style
everywhere.

Found via `codespell -q 3 -S ./git/ext/gitdb,./test/fixtures/reflog_master,./test/fixtures/diff_mode_only,./test/fixtures/reflog_HEAD`

Fix #971. Partly resolve #703.

If the hook doesn't have a file extension, then Windows won't know how
to run it and you'll get "[WinError 193] %1 is not a valid Win32
application". It's very likely that it's a shell script of some kind, so
use bash.exe (commonly installed via Windows Subsystem for Linux). We
don't want to run all hooks with bash because they could be .bat files.

Update tests to get several hook ones working. More work necessary to
get commit-msg hook working. The hook writes to the wrong file because
it's not using forward slashes in the path:
C:\Users\idbrii\AppData\Local\Temp\bare_test_commit_msg_hook_successy5fo00du\CUsersidbriiAppDataLocalTempbare_test_commit_msg_hook_successy5fo00duCOMMIT_EDITMSG

A regression that was introduced with d79d20d.

This isn't needed as git will replace this file atomicially,
hence we always see a fully written file when reading.

Only when writing we need to obtain a lock.

This reverts commit 53d94b8091b36847bb9e495c76bb5a3ec2a2fdb5.

The reason for the revert is that the commit in question introduced a
regression where certain modules, functions and classes that were
exposed before were no longer exposed.

See https://github.com/gitpython-developers/GitPython/pull/1352#issuecomment-932757204
for additional information.