GH-36973: [CI][Python] Archery linter integrated with flake8==6.1.0

[danepitkin]

Rationale for this change

Fix archery lint issues with new version of flake8 dependency.

What changes are included in this PR?

• Lint errors.

Are these changes tested?

Yes.

$ flake8 --version
6.1.0 (mccabe: 0.7.0, pycodestyle: 2.11.0, pyflakes: 3.1.0) CPython 3.11.4 on Darwin
$ archery lint --python
INFO:archery:Running Python formatter (autopep8)
INFO:archery:Running Python linter (flake8)
INFO:archery:Running Cython linter (cython-lint)
$

Are there any user-facing changes?

No

• Closes: [CI][Python] Archery linter is failing currently on main #36973

[github-actions]

⚠️ GitHub issue #36973 has been automatically assigned in GitHub to PR creator.

[danepitkin]

Works! https://github.com/apache/arrow/actions/runs/5727919670/job/15521373263?pr=36976#step:5:779

[raulcd]

I am not sure whether we should update the pre-commit version here:
https://github.com/apache/arrow/blob/main/.pre-commit-config.yaml#L40
Or remove it. I suppose it would use the latest one.
If we keep a version for pre-commit maybe we should pin it on archery too?
https://github.com/apache/arrow/blob/main/dev/archery/setup.py#L32

[danepitkin]

We should keep them in sync! I can pin both to 6.1.0 for now.

[raulcd]

Thanks @danepitkin , looks good to me! I'll merge once CI finishes.

[conbench-apache-arrow]

After merging your PR, Conbench analyzed the 5 benchmarking runs that have been run so far on merge-commit 5bb53c7.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about possible false positives for unstable benchmarks that are known to sometimes produce them.