Code

qa: fix additional LGTM alerts#1754

Code

Merged

dgw merged 11 commits intomastersopel-irc/sopel:masterfrom lgtm-tweakssopel-irc/sopel:lgtm-tweaksCopy head branch name to clipboardNov 18, 2019

+69-27

Commits

Commits on Nov 18, 2019

• isup: fix double-import of requests & command docstrings

  Show description for 94d9224
  dgw

  committed
  94d9224

  Copy full SHA for 94d9224

• contrib: remove unused import in makerpm.py

  dgw

  committed
  58cd948

  Copy full SHA for 58cd948

• plugins.handlers: fix double import with & without from

  dgw

  committed
  2ab58a5

  Copy full SHA for 2ab58a5

• module: ignore hard-to-refactor decorator duplication in LGTM

  dgw

  committed
  00b3377

  Copy full SHA for 00b3377

• qa: mismatched signature/use of overridden BaseValidated methods

  Show description for 05131d8
  dgw

  committed
  05131d8

  Copy full SHA for 05131d8

• irc.backends: silence SSL warnings in LGTM for now

  Show description for 3249ff7
  dgw

  committed
  3249ff7

  Copy full SHA for 3249ff7

• help: fix LGTM warning about URL sanitization

  Show description for afdaf5e
  dgw

  committed
  afdaf5e

  Copy full SHA for afdaf5e

• cli.utils: remove unnecessary named format args in get_many_text()

  dgw

  committed
  e0c5ece

  Copy full SHA for e0c5ece

• remind: appease LGTM with more magic methods

  Show description for bd02dc2
  dgw

  committed
  bd02dc2

  Copy full SHA for bd02dc2

• cli: add missing return 0s to various handlers

  Show description for ca39957
  dgw

  committed
  ca39957

  Copy full SHA for ca39957

• tools: make some things explicit to quiet LGTM warning

  dgw

  committed
  46f184e

  Copy full SHA for 46f184e