search

sopel-irc / sopel.chat

Website for Sopel
https://sopel.chat
Other
0 stars 11 forks source link

tutorials: Update memory tutorial to remove .contains() #19

Closed dgw closed 5 years ago

dgw commented 5 years ago

We're deprecating SopelMemory.contains() in version 7, and the actual example code already uses the in operator instead. in uses the magic __contains__() method, which is exactly what contains() calls, so… There's literally no difference. This should have been done a LONG time ago, like before the name change to Sopel (contains() has been a mere wrapper for __contains__() for nearly six years already). In fact, this can probably go on the live website whenever, without waiting for Sopel 7 to drop.

dgw commented 5 years ago

Decided to :shipit: as "part of" Sopel 6.6.8.