(ClosesGLolol/Supybot-Weather#1)
- Removed configuration option 'disableColoredTemp' (this will likely be readded in the form of a more global, strip-all-formatting option later)
- plugin.py: copy license into file heading
- Remove unused _moonphase function
- Rewrite almanac, astronomy, and forecast (merging multiple if statements into one)
- Update tests and output formatting
Signed-off-by: James Lu <GLolol1@hotmail.com>
- showImperialandMetric is dropped and now implied
- optlist handling is completely removed since it made already complex code even more so
- Lots of bugs introduced hopefully!
More specifically, test 'set', 'add', 'remove', 'clear', and 'list'. These are some relatively
complex functions that deal with: case-insensitive duplicate checking, requiring at least 2
channels per relay (auto-remove when a relay is below this threshold), adding/removing things
from existing relays, and more... Sometimes manual testing can't catch all of these complex
situations!
This fixes a bug where you could add duplicate channels using the 'add' command if the same
channel was given with differing cases (e.g. 'relaynext add new-relay channel1@net CHANNEl1@Net').
Also, add a notice when trying to remove a channel that doesn't exist in a relay.
- 'lastfm 'command is now split into subcommands, wrapping around one shared function (Closes#2).
- Use bolding for prettier formatting in output.
- Update command help/documentation to be more consistent (use "user", not "id" to refer to configured users).
- Rename variables named 'id' -> 'user', since the previous is collides with a built in function.
- Update tests accordingly.
New behavior: 'remove' without arguments removes entire relay, 'remove' with
channels specified will remove those channels in question. 'unset' is dropped.
Also update the README accordingly.
This isn't the most useful feature since a normal 'vlist' lookup can also
find source packages when 'config plugins.PkgInfo.archs' is configured
correctly.
