dgw / sopel-BombBot

Customized version of a custom version of a bomb module for Sopel
2 stars 1 forks source link

Settings refactor (resolves #14) #43

Closed dgw closed 6 years ago

dgw commented 6 years ago

Unified handling into a single callable. Takes "on"/"off" arguments instead of needing a dedicated command for each setting and state combination.

Renames and inverts the bombs_disabled setting to bombing_allowed. Includes migration routine that runs when performing checks during a call to .bomb (which writes the new setting and deletes the old). This migration might be changed before merging (e.g. to do it in bulk on load).