Fix TypeError when formatting with black 22.1.0+

[wlcx]

Resolve an TypeError issue caused by the return type of find_project_root changing in black 22.1.0.

Closes #29

[ccordoba12]

Hey @wlcx, thanks a lot for your help with this! Your fix is correct, but you also need to fix our tests because Black 22.1 dropped Python 2 support.

For that, please go to

python-lsp-black/tests/fixtures/target_version/pyproject.toml

Line 2 in 80403a7

target-version = ['py27']

and change it for

target-version = ['py39']

Then here

python-lsp-black/tests/test_plugin.py

Lines 175 to 178 in 80403a7

	def test_load_config_target_version():
	config = load_config(str(fixtures_dir / "target_version" / "example.py"))
	assert config["target_version"] == {black.TargetVersion.PY27}

please change PY27 to PY39.

That should be enough.

[ccordoba12]

@haplo, I think this requires an urgent 1.1.0 release.

[haplo]

@haplo, I think this requires an urgent 1.1.0 release.

Agreed. I'm reproducing the error and validating this fix.

@wlcx Will you be able to implement @ccordoba12's suggestion to fix the test suite? I can help with it if necessary, just let me know.

[wlcx]

Damn that was quick! On it now, also wrangling mypy errors.

[wlcx]

Do we need to retain backwards compatibility with prior black versions or can we require 22.1.0 as a minimum?

[haplo]

Do we need to retain backwards compatibility with prior black versions or can we require 22.1.0 as a minimum?

I would like to retain backwards-compatibility, some users might not be able to upgrade for some reason. Is something in particular making it difficult?

[haplo]

I would like to retain backwards-compatibility

This being said, I do believe python-lsp-black will only officially support the latest black version, so I will be adding a note about that in the README.

[wlcx]

Ok, think we're good to go, just need that first-time-contributor workflow approval

[haplo]

Tested fix with old and new black versions, both work. 👍

[haplo]

Thank you for your help @wlcx, I will prepare a release immediately.

[wlcx]

No worries, thanks for such an awesome and speedy response both!

[ccordoba12]

Thanks @haplo for your help with this!