dgw

Results 632 comments of dgw

mrw @Exirel adds something new to the 8.0.0 milestone: https://github.com/sopel-irc/sopel/assets/164140/39a0d31f-d226-4cc6-b6e4-6bb8aa22d583 I'm not super attached to the current site, other than it being nostalgic (the basic structure is the same as...

Hmm, this smells race-y, but on the server side. I don't think a client should have to be in the business of deciding which messages from the server are trustworthy....

I'm feeling pretty secure in _at minimum_ leaving this until after 8.0.0 based on testing elsewhere (non-Unreal-3.2 IRCds). `tfw` is an alt I used for testing; `GlaD0S` is network services:...

To follow up on [the discussion here](https://github.com/sopel-irc/sopel/pull/2563#issuecomment-1814968087) while bringing it back to the actual bug report: Maybe `coretasks` could keep track of pending WHO requests, but the `User` object shouldn't....

> @dgw suggested 9.0 for release of the initial implementation of a contextvars replacement, which I think would push the removal of `SopelWrapper` to 10.0, but we can discuss more...

Updated to shift the last two steps back a version (warning also in 8.1 now; removal in 9.0). I want this `str` arg gone. 😑

> unless we are very committed to a numeric priority, which I will admit is quite a bit more flexible. I'm very committed to making priorities more flexible, personally. Can't...

Unlike a relative path, an absolute path pointing to a symlink is still usable by downstream plugin code. If I `open()` a symlink I can read (or write) the real...

Might be reasonable for "this is a new nick I haven't seen before" to be handled somewhere in core before plugins ever have a chance to trigger, maybe a `@plugin.priority('high')`...

> I'm a -1 on this since it means doing work in the core that may be serving a plugin that isn't even enabled. I imagine most Sopel instances do...