comparison tests/test-check-pytype.t @ 48473:8843c9a8771b

pytype: stop excluding changegroup.py

The false positives that were detected seem to be related to what happens to the
variables after the local methods are used:

  File "/mnt/c/Users/Matt/hg/mercurial/changegroup.py", line 353, in ondupchangelog:
      No attribute 'append' on range [attribute-error]
    In Union[List[nothing], range]
  File "/mnt/c/Users/Matt/hg/mercurial/changegroup.py", line 357, in onchangelog:
      No attribute 'update' on None [attribute-error]

Differential Revision: https://phab.mercurial-scm.org/D11912

48472:a4d8de93023c

I can tell, so let's just hide those files from it for now. We should
endeavor to empty this list out over time, as some of these are
probably hiding real problems.

mercurial/bundlerepo.py       # no vfs and ui attrs on bundlerepo
mercurial/changegroup.py      # mysterious incorrect type detection
mercurial/chgserver.py        # [attribute-error]
mercurial/cmdutil.py          # No attribute 'markcopied' on mercurial.context.filectx [attribute-error]
mercurial/context.py          # many [attribute-error]
mercurial/copies.py           # No attribute 'items' on None [attribute-error]
mercurial/crecord.py          # tons of [attribute-error], [module-attr]

TODO: use --no-cache on test server?  Caching the files locally helps during
development, but may be a hinderance for CI testing.

  $ pytype -V 3.6 --keep-going --jobs auto mercurial \
  >    -x mercurial/bundlerepo.py \
  >    -x mercurial/changegroup.py \
  >    -x mercurial/chgserver.py \
  >    -x mercurial/cmdutil.py \
  >    -x mercurial/context.py \
  >    -x mercurial/copies.py \
  >    -x mercurial/crecord.py \

48473:8843c9a8771b

I can tell, so let's just hide those files from it for now. We should
endeavor to empty this list out over time, as some of these are
probably hiding real problems.

mercurial/bundlerepo.py       # no vfs and ui attrs on bundlerepo
mercurial/chgserver.py        # [attribute-error]
mercurial/cmdutil.py          # No attribute 'markcopied' on mercurial.context.filectx [attribute-error]
mercurial/context.py          # many [attribute-error]
mercurial/copies.py           # No attribute 'items' on None [attribute-error]
mercurial/crecord.py          # tons of [attribute-error], [module-attr]

TODO: use --no-cache on test server?  Caching the files locally helps during
development, but may be a hinderance for CI testing.

  $ pytype -V 3.6 --keep-going --jobs auto mercurial \
  >    -x mercurial/bundlerepo.py \
  >    -x mercurial/chgserver.py \
  >    -x mercurial/cmdutil.py \
  >    -x mercurial/context.py \
  >    -x mercurial/copies.py \
  >    -x mercurial/crecord.py \