Micha Reiser

Results 804 comments of Micha Reiser

> It only appears when the cache isn't used: This is because Ruff tries to create the fix on the first run (and fails) but then caches the diagnostic without...

> This lint is current in the experimental state, so it won't be enabled by default ...? That's correct. But the intent would be to stabilize the rule eventually (ideally...

I discussed this internally and @charliermarsh is fine with adding new pylint rules, even if they are very opinionated.

I think this is the same as https://github.com/astral-sh/ruff/pull/10431. We need to decide which of the two PRs we want to move forward.

Thank you for contributing to this new rule, and I am sorry that it took so long to decide. We agree that the rule itself is useful, and we're open...

This is an interesting usage of the new override system and should be supported. However, It may not have the desired result depending on your use case: ``` pyproject.toml tests\...

Hy @Beliavsky. I'll close this as resolved. Feel free to reopen or comment if you're still facing the issue.

> Black seems to leave this code unmodified in preview. Not entirely. Black hugs the parentheses, but formats all entries on a single line ```python self._edits.append({ "operation": "update", "id": index,...

It seems they changed this between Stable and Main. Stable gives you the formatting I shared. Main leaves it as is. I don't like the change. Makes it rather magic...

Note: The pattern where the "huggable" fully fits on its own line seems somewhat common in Airflow's codebase https://github.com/apache/airflow/pull/37355