Andrey

Results 75 comments of Andrey

> > Hi @jdolitsky. The order and precedence is documented with helm help install, and is available here: https://github.com/helm/helm/blob/03ccd5dea2516e5da275653aa4485064377a7fd5/cmd/helm/install.go#L46-L68 > > Documented here: https://helm.sh/docs/helm/helm_install/ > > is there another location...

> Unless there's a substantial measurable performance issue, I would leave this as updating the whole universe. And unless you're doing a lot of meter updates, it's probably not a...

Plugins that are useful for me are: - [flake8-eradicate](https://pypi.org/project/flake8-eradicate/1.2.0/) remove commented-out code, quite useful to keep the repo in shape. - [flake8-pytest-style](https://pypi.org/project/flake8-pytest-style/) makes all tests look consistent My personal list...

To run flake8 with plugins you need to install them first. This line in the contribution guide don't mention this. Since flake8 is already included in `pre-commit run --all-files ...`...

> Maybe docstrings? https://github.com/PyCQA?q=flake8- But I am sure a ton of existing files do not have docstrings. Flake is pretty flexible with ignoring files with specific errors. You could suppress...

> flake8-use-pathlib sounds awesome (although I have never seen it in action). I have some concerns about it. This code looks super unusual to me. ```python with Path(filename).open() as f:...

Sample PR on how the code should look to conform the flake8-docstrings. https://github.com/TheAlgorithms/Python/pull/7821 It would be nice to understand the check that we plan to keep and which one we...

I'd say it is closed a bit too earlier. I haven't submitted the PR with annotation-docstring yet. I am planning to do it today in the evening, so no need...

I have a free evening today, could I help with the review?

> What do I*, F*, Turn*, and Total* indicate? - Issued succesfuly this turn - Failed this turn - Total this turn - Total for the current session `*` are...