dgw

Results 632 comments of dgw

OK! The good news is, fixing the screwed-up tree/source views was pretty simple. Bad news is that display of branch coverage is still partially broken on Coveralls with this new...

Hm, still does not seem to report `branch = True` results correctly using the `.coverage` file report type. 🤔 There is also a "Drifted build" warning, but that should be...

@afinetooth Thanks for the rundown! We aren't so attached to the coverage _diff_ of each patch being perfectly accurate, and I suspect we will keep the current CI setup (build...

I've run into intermittent problems with restarting using the typical `sopel restart -c configname` spelling, too. Hard to troubleshoot as sometimes it works fine, sometimes not. Could be related to...

Can confirm, if I start a Sopel instance (with the normal `sopel -c configname` command line) and later restart it _once_ it's fine, then the restart fails with the `FileNotFoundError`...

There's a thought. Maybe with a `Proceed (Y/n)?` prompt like you get for `pip uninstall`.

> It adds a dependency on `scramp` (~29kb), which could be made optional if we're picky. Avoiding/trying to eliminate dependencies in the current era of Sopel (can't speak for past...

Wasn't saying anything about this specific library, haha, just dependencies in general. Sure, I've seen ProgVal around a lot, and would probably have picked that one had I written this...

@progval @tlocke Have your concerns all been addressed? I'm going through some of our older open PRs and this one looks like it should be either ready or very nearly...

@tlocke Thanks. I saw a traceback example and assumed you were advising a fix for something still in the patch, or modifying the version range. @Exirel It might not be...