Alexander Burchenko

Results 24 comments of Alexander Burchenko

It's a very simple update - two files need changing: requirements.pep8.txt ``` flake8==5.0.4 + flake8-use-fstring==1.4 ``` tox.ini ``` [testenv:pep8] deps = -r{toxinidir}/requirements.pep8.txt - commands = flake8 + commands = flake8...

I have implemented what I suggested. Please do a PR review. I may recommend extending f-string to the rest of the code in the future.

@PawelLipski Give me a couple of days - I'll see what the problem is

@PawelLipski issue still open? I ran the tests and didn't get this error.

there is a very simple solution instead of ``` - name: Upload coverage report to codecov run: | bash

@darrenburns, this issue has already been implemented a long time ago. However, it looks like it's time to close it.

Simply changing the class from [Text](https://rich.readthedocs.io/en/latest/reference/text.html#rich.text.Text) to [Markdown](https://rich.readthedocs.io/en/latest/reference/markdown.html#rich.markdown.Markdown) won't help. Parameter `title` of the [Rule](https://rich.readthedocs.io/en/latest/reference/rule.html?highlight=rule#rich.rule.Rule) class has the type `str | Text`. Argument of type `Markdown` cannot be assigned to...

@darrenburns, this issue has already been implemented a long time ago. However, it looks like it's time to close it.

Hi @phil-r I'm ready to solve this issue

I analysed the code with `black --check -t py37 --diff --color` **black** will change: - quotes from `'` to `"`. - `lengthen' some of the lines that are now wrapped...