Remove pylint configuration file

[doolio]

Project now uses ruff for linting so this config file is obsolete.

[ccordoba12]

It seems pylintrc is used in a couple of tests. So you need to move it to the tests directory instead and update the respective tests.

[doolio]

But is that not because we still install pylint and pycodestyle? Should these have not been removed as pyflakes and black were?

python-lsp-server/.github/workflows/static.yml

Lines 38 to 42 in fc2851a

	# If we don't install pycodestyle, pylint will throw an unused-argument error in pylsp/plugins/pycodestyle_lint.py:72
	# This error cannot be resolved by adding a pylint: disable=unused-argument comment ...
	- run: |
	pip install -e .[pylint,pycodestyle]
	pip install ruff

cc: @tkrabel

[tkrabel]

But is that not because we still install pylint and pycodestyle? Should these have not been removed as pyflakes and black were?

python-lsp-server/.github/workflows/static.yml

Lines 38 to 42 in fc2851a

	# If we don't install pycodestyle, pylint will throw an unused-argument error in pylsp/plugins/pycodestyle_lint.py:72
	# This error cannot be resolved by adding a pylint: disable=unused-argument comment ...
	- run: |
	pip install -e .[pylint,pycodestyle]
	pip install ruff

cc: @tkrabel

I came to the conclusion that pylint is still used in some tests because of the pylint plugin. That’s why it needs to be installed, requiring also pycodestyle. I won’t resist removing that if you find a way to make all tests pass :)

[doolio]

I guess it makes sense we install pylint to test the pylint plugin but then I don't understand then why we no longer install pyflakes to test the pyflakes plugin (and perhaps the flake8 plugin)? Or the other tools (autopep8, mccabe, pydocstyle, rope and yapf for their respective plugins for that matter? I'm still a beginner so forgive my ignorance here.

In any case, I think we can still remove this config file and have a [tool.pylint] section in the pyproject.toml (which pylint supports) at the very least. This way we have only the one config file to maintain.

[ccordoba12]

@doolio, I think you're misunderstanding things a bit. If you look at one of the failing tests

you'll see its workspace is set at the root of the repo to get access to its pylintrc file. So, you need to move that file to another place inside our tests directory and update that test accordingly.

[doolio]

I think you're misunderstanding things a bit.

Very likely so I appreciate you taking the time. These tests are failing because I have removed the .pylintrc with this PR and presumably do not fail if it is present, right? So I think pylint searches up the directory tree until it finds its configuration file whether that be a .pylintrc or the pyproject.toml though the former takes precedence. Therefore, I think if I add the pylint configuration to the pyproject.toml the tests should pass If they were passing before, no?

My aim with this PR was to remove .pylintrc so we have less configuration to maintain.

[ccordoba12]

Therefore, I think if I add the pylint configuration to the pyproject.toml the tests should pass If they were passing before, no?

That should be the case, yes. But I don't think it's necessary because we're not using Pylint for the project anymore. That's why I propose to move the current pylintrc to our test folder and update the failing tests accordingly.