flake8
-
rbtools/commands/land.py (Diff revision 1) -
-
Review Request #13599 — Created March 1, 2024 and submitted
This removes the last of the stuff slated for removal in RBTools 4.0
(which we didn't actually remove in 4.0). The only silly thing here was
the submit_as argument to guess_existing_review_request, which had been
marked as required for 4.0 but we weren't actually passing in values
from most callsites.
rbt land
, rbt post
, and rbt stamp
.Summary | ID |
---|---|
c24f091a3f69fd230179e8f0a5eae3a60083cdef |
Description | From | Last Updated |
---|---|---|
undefined name 'Optional' Column: 22 Error code: F821 |
reviewbot | |
undefined name 'Optional' Column: 16 Error code: F821 |
reviewbot | |
undefined name 'Optional' Column: 31 Error code: F821 |
reviewbot | |
Since we're fetching each of these many times, and asserting on them, let's just pull them out into locals, do … |
chipx86 | |
Same applies here. |
chipx86 | |
Same applies here. |
chipx86 | |
Also, all arguments are now keyword-only. |
chipx86 | |
Also, keyword-only args. |
chipx86 |
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+324 -354) |
rbtools/commands/land.py (Diff revision 2) |
---|
Since we're fetching each of these many times, and asserting on them, let's just pull them out into locals, do the assert, and then use those below.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 3 (+398 -394) |