Rule E203 conflicts with formatter

[ofek]

Running the formatter on

release_lines = history_file_lines[history_file_lines.index('## Unreleased') + 1: -1]

produces

release_lines = history_file_lines[history_file_lines.index('## Unreleased') + 1 : -1]

which then fails the rule

scripts\utils.py:22:85: E203 Whitespace before ':'

Is the fix simply to document that here? https://docs.astral.sh/ruff/formatter/#conflicting-lint-rules

[charliermarsh]

I will add it to that list, though there's an open issue to fix the rule to allow this: #7259.

[charliermarsh]

I decided to just fix it instead.

[ofek]

Oh nice that's much better, thanks!

[JohannesBuchner]

I am still getting this unwanted behaviour with ruff-0.9.3:

$ cat > foo.py
lanes[laneid + 1:]
$ ruff format foo.py
1 file reformatted
$ cat foo.py
lanes[laneid + 1 :]

[dhruvmanila]

@JohannesBuchner That's an expected behavior. What the original issue was pointing out is that E203 rule is removing the whitespace before the : operator while the formatter adds it - causing the conflicting behavior between the linter and the formatter.

[JohannesBuchner]

@dhruvmanila Thanks for the quick response. flake8 gives the error "E203 whitespace before ':'" with the reformatted output. I see from psf/black#315 that they claim that this is an issue of pep8. There is help at https://ichard26-testblackdocs.readthedocs.io/en/refactor_docs/compatible_configs.html for configuring flake8. Thanks.