Merge lp:~raharper/curtin/trunk.fix-tox-errors into lp:~curtin-dev/curtin/trunk
Status: | Merged |
---|---|
Merged at revision: | 366 |
Proposed branch: | lp:~raharper/curtin/trunk.fix-tox-errors |
Merge into: | lp:~curtin-dev/curtin/trunk |
Diff against target: |
56 lines (+7/-7) 2 files modified
curtin/commands/main.py (+5/-5) tox.ini (+2/-2) |
To merge this branch: | bzr merge lp:~raharper/curtin/trunk.fix-tox-errors |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Server Team CI bot | continuous-integration | Approve | |
Scott Moser | Pending | ||
Review via email: mp+290531@code.launchpad.net |
Description of the change
curtin: fix tox errors in commands/main.py
Unexpectedly, pylint now blows up with the following errors:
E:208,13: Instance of 'argv' has no 'reportstack' member (but some types could
not be inferred) (maybe-no-member)
E:209,18: Instance of 'argv' has no 'func' member (but some types could not be
inferred) (maybe-no-member)
This code has been in tree for a very long time, this only shows up when
we added pylint to the tox run (revno 335). This should have always failed.
With that said, the fix is simple. In main(), we define args to be sys.argv
but it's not used until later we re-assign args to be something else from
argparser. Removing that useless initialization as a argv type resolves
the lint error.
PASSED: Continuous integration, rev:366 /server- team-jenkins. canonical. com/job/ curtin- ci/197/ /server- team-jenkins. canonical. com/job/ generic- update- mp/194/ console
https:/
Executed test runs:
None: https:/
Click here to trigger a rebuild: /server- team-jenkins. canonical. com/job/ curtin- ci/197/ rebuild
https:/