view tests/test-check-pylint.t @ 49798:a51328ba33ca

ui: split the `default` arg out of **kwargs for the internal prompt method

This arg was required anyway, based on how it was accessed.  Having it separate
allows it to be typed though, and this will simplify things for the callers- if
a non-None `default` is passed, the return can never be None.  That can be
expressed with `@overload` when the arg can be typed, but that's not possible
when it is rolled up in **kwargs.

The default value is simply copied from the public `prompt()` above it.

#require test-repo pylint

Run pylint for known rules we care about.
-----------------------------------------

There should be no recorded failures; fix the codebase before introducing a
new check.

Current checks:
- W0102: no mutable default argument

  $ touch $TESTTMP/fakerc
  $ pylint --rcfile=$TESTTMP/fakerc --disable=all \
  >   --enable=W0102,C0321 \
  >   --reports=no \
  >   --ignore=thirdparty \
  >   mercurial hgdemandimport hgext hgext3rd | sed 's/\r$//'
  Using config file *fakerc (glob) (?)
   (?)
  ------------------------------------* (glob) (?)
  Your code has been rated at 10.00/10* (glob) (?)
   (?)