Merge pull request #103 from xenserver-next/pin-tox==4.5.1,update-act… #494
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
Annotations
4 errors, 2 warnings, and 10 notices
test (3.6, ubuntu-22.04)
The run was canceled by @bernhardkaindl.
|
test (2.7, ubuntu-22.04)
The operation was canceled.
|
test (3.8, ubuntu-22.04)
The run was canceled by @bernhardkaindl.
|
test (3.8, ubuntu-22.04)
The operation was canceled.
|
test (3.6, ubuntu-22.04):
tests/test_ifrename_dynamic.py#L24
unclosed file <_io.TextIOWrapper name=16 mode='r' encoding='ANSI_X3.4-1968'>
|
test (2.7, ubuntu-22.04)
The support for python 2.7 will be removed on June 19. Related issue: https://github.com/actions/setup-python/issues/672
|
pylint C0123: unidiomatic-typecheck:
xcp/version.py#L85
Convention: Use isinstance() rather than type() for a typecheck.
|
pylint C0123: unidiomatic-typecheck:
xcp/version.py#L86
Convention: Use isinstance() rather than type() for a typecheck.
|
pylint R1710: inconsistent-return-statements:
xcp/xmlunwrap.py#L79
Refactor: Either all return statements in a function should return an expression, or none of them should.
|
pylint C0325: superfluous-parens:
xcp/pci.py#L85
Convention: Unnecessary parens after 'not' keyword
|
pylint C0325: superfluous-parens:
xcp/pci.py#L90
Convention: Unnecessary parens after 'not' keyword
|
pylint C0325: superfluous-parens:
xcp/pci.py#L95
Convention: Unnecessary parens after 'not' keyword
|
pylint C0325: superfluous-parens:
xcp/mount.py#L48
Convention: Unnecessary parens after 'assert' keyword
|
pylint R1724: no-else-continue:
xcp/net/ifrename/logic.py#L239
Refactor: Unnecessary "else" after "continue", remove the "else" and de-indent the code inside it
|
pylint R1724: no-else-continue:
xcp/net/ifrename/logic.py#L259
Refactor: Unnecessary "else" after "continue", remove the "else" and de-indent the code inside it
|
pylint R1724: no-else-continue:
xcp/net/ifrename/static.py#L228
Refactor: Unnecessary "elif" after "continue", remove the leading "el" from "elif"
|