Update pre-commit versions

[cdce8p]

https://black.readthedocs.io/en/stable/change_log.html#id1

https://flake8.pycqa.org/en/latest/release-notes/7.1.2.html
https://flake8.pycqa.org/en/latest/release-notes/7.2.0.html
https://flake8.pycqa.org/en/latest/release-notes/7.3.0.html

https://github.com/PyCQA/isort/releases/tag/6.0.0
https://github.com/PyCQA/isort/releases/tag/6.0.1

[ohemorange]

am I missing something in the black changelog? I don't see a note about it supporting 3.14

[cdce8p]

am I missing something in the black changelog? I don't see a note about it supporting 3.14

Seems it was added as part of psf/black#4720. Note that it isn't necessary that black supports all (new) features of 3.14 just yet as those can't be used until 3.13 is dropped anyway.

All that was necessary is that py314 is now an allowed value for black.

[ohemorange]

I see. Looks like when they officially announced 3.13 support, they tested it in various places first. https://github.com/psf/black/pull/4436/files That has not yet been done for 3.14, so I'd say black 3.14 support is still incidental and unofficial. Personally, I would prefer to wait until they announce stable support for 3.14.

[cdce8p]

Personally, I would prefer to wait until they announce stable support for 3.14.

Not sure it really matters in the context of this PR. It's still years until we'd be able to use Template strings here for example. Technically, all that's really needed with target-version often times is to specified the lowest supported version.

Anyway, there is also the option to get rid of it completely if we adopt poetry-core v2 which supports project.requires-python. Opened #242 for that. Let me know what you think.

[ohemorange]

I like that plan. Looks like #242 got auto-assigned, I moved it to my queue and will take a look but at first glance I like that approach.

[bmw]

lgtm!