dorschw comments

Results 5 comments of


                                            dorschw

LGTM.com - false positive: `import os` counts as syntax error

Cool, thanks On Mon, Aug 1, 2022, 18:55 Arthur Baars ***@***.***> wrote: > @dorschw > > The import os line looks indeed fine. However, > Python 3.8.10 appears to give...

Support baselines, i.e., ignore existing errors for incremental adoption

As (temporary?) solution, I just released Riff [PyPi](https://pypi.org/project/riff) / [GitHub](https://github.com/dorschw/riff), Riff wraps Ruff, runs git diff, checks for modified lines, only fails if errors were found in modified lines. Works...

Support pre-commit hooks

> In your setup are you assuming that the dependencies are available and a configuration file is present? When using `system` hooks, pre-commit becomes a task runner, assuming the entry...

toml-sort breaks ruff config by removing tool.ruff. prefix from sections

adding a `[tool.ruff.lint]` header fixed it for me: ```diff [tool.ruff] - lint.ignore = ["F403"] + [tool.ruff.lint] + ignore = ["F403"] ``` not sure if it's just a workaround, **or** whether...

Pre-commit hook suggestions

Hey, this can be useful for us, any ETA for a merge? (I saw the discussion)