dgw

Results 632 comments of dgw

> * I did see some errors when running `mypy` because `types-urllib3` was not installed. I'm not sure if that's missing from `dev-requirements.txt` or if my local environment was in...

Contributing that 973a489355540d68b95db01a49e983ac7a740bcc on Python 3.10.7 exhibits a related behavior in `sopel-spongemock`: > \ .smock _italic mocking_ > \ iTaLIc MoCkiNg This command uses `trigger.group(2)`. Note that the result is...

#2519 made unpinning `urllib3` possible, and the upstream issue in `vcrpy` has been "closed". But we're still doing Kludgy Things related to both packages: https://github.com/sopel-irc/sopel/blob/973a489355540d68b95db01a49e983ac7a740bcc/dev-requirements.txt#L20-L22 I will retitle this issue...

This has been at the back of my mind since it was opened, and I finally have something meaningful to say about it. I don't think PR authors should be...

> > As I see it, the "unreleased" changelog that @Exirel suggested should be maintained in a long-running branch, and updated when patches land. > > Why a separate branch?...

The Coveralls action is supposed to be more or less a drop-in replacement for the `coveralls` CLI command, or so I thought. Will have to dig deeper into how the...

For fun, I tried to run `coveralls report --debug --dry-run` on my local `.coverage` file, after running `coverage run -m pytest`. `coveralls` correctly detected the coverage format (`python`) and filename...

Hmm, this changed some things and I think it's considering some things covered that shouldn't be, hence the +0.6% bump. For example, some information about branches seems to be lost...

Filed coverallsapp/coverage-reporter#109 for the `--branch` coverage weirdness, and we'll see where that goes before I spend more time on this.

The missing link here was generating an XML coverage report, instead of LCOV. https://github.com/coverallsapp/coverage-reporter/issues/109#issuecomment-1807646155 However, I do need to still track down a problem with the tree view after switching....