Details

mercurial/commands.py
1124	This one is sad. I think this can be sorted out by replacing the `try:/finally:` with a context manager. (Can send a patch, if it sounds good to you.)

durin42 added a child revision: D7385: debugcommands: suppress import errors for pytype.Nov 14 2019, 4:10 PM

durin42 removed a child revision: D7295: pytype: add a (very slow) test that executes pytype.

durin42 updated this revision to Diff 18099.

durin42 added inline comments.Nov 14 2019, 4:15 PM

mercurial/commands.py
1124	By all means!

dlax mentioned this in D7430: bisect: replace try:/finally: by a "restore_state" context manager.Nov 15 2019, 5:13 AM

D7430 makes this changes unnecessary I think.

This revision now requires changes to proceed.Nov 15 2019, 6:08 AM

I still want to keep the annotations I added. :)

dlax mentioned this in rHGf37da59a36d9: bisect: replace try:/finally: by a "restore_state" context manager.Nov 15 2019, 12:24 PM

durin42 updated this revision to Diff 18162.Nov 15 2019, 12:31 PM

dlax added inline comments.Nov 15 2019, 12:40 PM

mercurial/commands.py
4746	`revs` is always a `smartset.baseset` per af9c73f26371 so there should be no attribute error. Or is it because `logcmdutil.getlinerangerevs()` has no type annotation (whereas `logcmdutil.getrevs()` has some)?

durin42 updated this revision to Diff 18171.Nov 15 2019, 4:55 PM

durin42 edited parent revisions, added: D7297: cleanup: remove now-obsolete wrong-arg-type annotations; removed: D7383: state: add a pytype annotation.Nov 19 2019, 11:14 AM

durin42 added a child revision: D7295: pytype: add a (very slow) test that executes pytype.

durin42 edited parent revisions, added: D7296: pycompat: kludge around pytype being confused by __new__; removed: D7297: cleanup: remove now-obsolete wrong-arg-type annotations.Nov 19 2019, 1:20 PM

mharbison72 added a child revision: D7515: pytype: [WIP] suppress various warnings inline to get a clean run.Nov 23 2019, 7:47 PM

What's up on this ? It seemed on a good track, but I don't think it landed. @dlax I think you offer to use a context manager got a warm welcome, I would says, go ahead with it.

In D7384#118739, @marmoute wrote:

What's up on this ? It seemed on a good track, but I don't think it landed. @dlax I think you offer to use a context manager got a warm welcome, I would says, go ahead with it.

The context manager got in through D7430.

mercurial/commands.py
4741	I think this comment should be removed since the `# pytype: disable` got removed in the last version of the diff.

There don't seems to be any remaininf feedback on this.

There seems to have been no activities on this Diff for the past 3 Months.

By policy, we are automatically moving it out of the need-review state.

Please, move it back to need-review without hesitation if this diff should still be discussed.

:baymax:need-review-idle:

This revision now requires changes to proceed.Jun 8 2020, 1:34 PM

Herald added a subscriber: mercurial-patches. · View Herald TranscriptJun 8 2020, 1:34 PM

mharbison72 mentioned this in D10165: typing: fix a suppression directive that was mangled by black formatting.Mar 12 2021, 1:12 PM

mharbison72 mentioned this in D10236: commands: necessary annotations and assertions to pass pytype.Mar 19 2021, 1:58 AM

mharbison72 mentioned this in rHGb26f9560f40d: commands: necessary annotations and assertions to pass pytype.Mar 22 2021, 2:29 PM

Diff	ID	Description	Created	Lint	Unit
Base		Base
Diff 1	18071		Nov 13 2019, 10:51 PM	★	★
Diff 2	18099		Nov 14 2019, 4:09 PM	★	★
Diff 3	18162		Nov 15 2019, 12:31 PM	★	★
Diff 4	18171		Nov 15 2019, 4:55 PM	★	★

Status	Author	Revision
Closed	durin42	D7406 scmutil: convert status data object from a tuple to an attrs (API)
Closed	durin42	D7405 perf: bool() elements of dirstate.status return instead of len()
Closed	durin42	D7404 tags: use field names instead of field numbers on scmutil.status
Closed	durin42	D7403 subrepo: use field names instead of field numbers on scmutil.status
Closed	durin42	D7402 revset: use field names instead of field numbers on scmutil.status
Closed	durin42	D7401 patch: use field names instead of field numbers on scmutil.status
Closed	durin42	D7400 logcmdutil: use field names instead of field numbers on scmutil.status
Closed	durin42	D7399 context: use field names instead of field numbers on scmutil.status
Closed	durin42	D7398 commands: use field names instead of field numbers on scmutil.status
Closed	durin42	D7397 cmdutil: convert terse status result back to an scmutil.status
Closed	durin42	D7396 cmdutil: use field names instead of field numbers on scmutil.status
Closed	durin42	D7395 uncommit: use field names instead of field numbers on scmutil.status
Closed	durin42	D7394 transplant: use field names instead of field numbers on scmutil.status
Closed	durin42	D7393 split: use field names instead of field numbers on scmutil.status
Closed	durin42	D7392 mq: use field names instead of field numbers on scmutil.status
Closed	durin42	D7391 hgk: use field names instead of field numbers on scmutil.status
Closed	durin42	D7390 extdiff: use field names instead of field numbers on scmutil.status
Closed	durin42	D7294 cmdutil: suppress bogus pytype errors
Closed	durin42	D7293 changegroup: suppress pytype error that's wrong
Closed	durin42	D7292 changegroup: avoid shadowing a set with an int
Closed	durin42	D7291 bundle2: more pytype suppressions around bytestring
Closed	durin42	D7290 branchmap: pytype is confused about bytestr
Closed	durin42	D7410 extensions: suppress a pytype failure due to a typeshed bug
Closed	durin42	D7409 dispatch: add some assertions to give pytype a helping hand
Closed	durin42	D7408 extensions: hide two confusing import statements from pytype
Closed	durin42	D7407 debugcommands: add assertions to convince pytype peer is not None
Closed	durin42	D7385 debugcommands: suppress import errors for pytype
Closed	durin42	D7383 state: add a pytype annotation
Closed	durin42	D7382 logcmdutil: add a type annotation
Closed	durin42	D7381 cmdutil: add a pytype annotation to help out some callsites
Abandoned	durin42	D7297 cleanup: remove now-obsolete wrong-arg-type annotations
Abandoned	durin42	D7380 encoding: define per-use identity functions only in typechecking mode
Changes Planned	mharbison72	D7515 pytype: [WIP] suppress various warnings inline to get a clean run
Needs Revision	durin42	D7295 pytype: add a (very slow) test that executes pytype
Needs Revision	durin42	D7384 commands: necessary annotations and suppresssions to pass pytype
Changes Planned	durin42	D7296 pycompat: kludge around pytype being confused by __new__
Closed	durin42	D7379 encoding: fix bad type annotation
Closed	durin42	D7389 branchmap: annotate constructor type for branchcache
Closed	durin42	D7378 debugcommands: don't shadow the error module
Closed	durin42	D7289 branchmap: correctly set()-ify list argument

commands: necessary annotations and suppresssions to pass pytype
Needs RevisionPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 18171

mercurial/commands.py

commands: necessary annotations and suppresssions to pass pytypeNeeds RevisionPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 18171

mercurial/commands.py

commands: necessary annotations and suppresssions to pass pytype
Needs RevisionPublic
Actions

Revision Contents
Changeset List