Skip to content

Commit 9a9fe4d

Browse files
kartbennordicjm
authored andcommitted
[nrf fromtree] scripts: ci: check_compliance: no need for lxml anymore
Remove an old workaround requiring lxml to be present as junitparser was not working with xml.etree.ElementTree until version 3.0.0. Signed-off-by: Benjamin Cabé <[email protected]> (cherry picked from commit db47c89)
1 parent e6b70b6 commit 9a9fe4d

File tree

2 files changed

+1
-9
lines changed

2 files changed

+1
-9
lines changed

scripts/ci/check_compliance.py

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2288,14 +2288,6 @@ def _main(args):
22882288
def main(argv=None):
22892289
args = parse_args(argv)
22902290

2291-
try:
2292-
# pylint: disable=unused-import
2293-
from lxml import etree
2294-
except ImportError:
2295-
print("\nERROR: Python module lxml not installed, unable to proceed")
2296-
print("See https://github.com/weiwei/junitparser/issues/99")
2297-
return 1
2298-
22992291
try:
23002292
n_fails = _main(args)
23012293
except BaseException:

scripts/requirements-build-test.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,4 +18,4 @@ pytest
1818
mypy
1919

2020
# used for JUnit XML parsing in CTest harness
21-
junitparser
21+
junitparser>=3.0.0

0 commit comments

Comments
 (0)