fix: cli #78
fix: cli #78
Conversation
|
Формат PR пойдет, только надо добавить ссылку на issue. |
| rich = "==13.9.4" | ||
| click = ">=8.2.1" | ||
| dacite = "==1.9.2" | ||
| pysatl-criterion = {path = "./pysatl_criterion"} |
There was a problem hiding this comment.
Оставлю здесь заметку на потом.
| lines-after-imports = 2 | ||
|
|
||
| [tool.flake8] | ||
| ignore = ["E121", "E123", "E126", "E24", "E203", "E704", "W503", "W504"] |
There was a problem hiding this comment.
flake8 линтер и ruff линтер, делают одно и тоже. В PySATL'e используют ruff
There was a problem hiding this comment.
Я про сами коды варнингов (например, https://www.flake8rules.com/rules/E121.html).
Имхо, стоит их добавить в соответствующий раздел для ruff.
There was a problem hiding this comment.
Так даже не было же в зависимостях flake8, это мертвый код изначально?
There was a problem hiding this comment.
Хотя он есть в pre-commit'e, но я не очень понимаю как это должно работать
There was a problem hiding this comment.
Ладно, пока так оставим.
При необходимости добавлю отдельным коммитом, пускай ветка остаётся (и заметка здесь).
There was a problem hiding this comment.
@alex98247, думаю, можно мерджить (только ветку оставь, пожалуйста).
Summary
Fixed CLI.
Solve the issue: #76
Quick changelog
pyproject.tomlfor Poetry compatibility.shared.pyto centralize common CLI dependencies.cli.pyandconfigure.pyto use the new shared module, removing direct dependencies.__main__.pyto provide a standard executable entry point for the package.What's new?
This PR fixes known bugs with CLI dependencies.