Skip to content

Commit 2283bc5

Browse files
committed
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]>
1 parent 8abbfb6 commit 2283bc5

File tree

3 files changed

+2
-10
lines changed

3 files changed

+2
-10
lines changed

scripts/ci/check_compliance.py

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2235,14 +2235,6 @@ def _main(args):
22352235
def main(argv=None):
22362236
args = parse_args(argv)
22372237

2238-
try:
2239-
# pylint: disable=unused-import
2240-
from lxml import etree
2241-
except ImportError:
2242-
print("\nERROR: Python module lxml not installed, unable to proceed")
2243-
print("See https://github.com/weiwei/junitparser/issues/99")
2244-
return 1
2245-
22462238
try:
22472239
n_fails = _main(args)
22482240
except BaseException:

scripts/requirements-build-test.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,4 +21,4 @@ mypy
2121
mock>=4.0.1
2222

2323
# used for JUnit XML parsing in CTest harness
24-
junitparser
24+
junitparser>=3.0.0

scripts/requirements-compliance.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
# zephyr-keep-sorted-start
55
clang-format>=15.0.0
66
gitlint
7-
junitparser>=2
7+
junitparser>=3.0.0
88
lxml>=5.3.0
99
pykwalify
1010
pylint>=3

0 commit comments

Comments
 (0)