tox: fix lint target on python 3.13

Update all packages used in the lint target to their latest release.
Explicitly bump the ubuntu image version in all GitHub action jobs. Fix
errors reported by new pylint/black versions.

Signed-off-by: Robin Jarry <[email protected]>

Author: rjarry

.github/workflows/ci.yml

@@ -7,7 +7,7 @@ on: [push, pull_request]
 
 jobs:
   lint:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     steps:
       - uses: actions/checkout@v3
       - uses: actions/setup-python@v4
@@ -36,11 +36,11 @@ jobs:
       - run: make check-commits
 
   test:
-    runs-on: ubuntu-20.04
+    runs-on: ubuntu-24.04
     strategy:
       matrix:
         include:
-          - python: 3.9
+          - python: "3.9"
             toxenv: py39
           - python: "3.10"
             toxenv: py310
@@ -56,7 +56,7 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-python@v4
         with:
-          python-version: ${{ matrix.python }}
+          python-version: "${{ matrix.python }}"
       - uses: actions/cache@v3
         with:
           path: ~/.cache/pip
@@ -86,7 +86,7 @@ jobs:
   deploy:
     needs: [lint, test]
     if: github.event_name == 'push' && startsWith(github.ref, 'refs/tags/v')
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.03
     steps:
       - uses: actions/checkout@v3
       - uses: actions/setup-python@v4

libyang/schema.py

@@ -1542,9 +1542,7 @@ def defaults(self) -> Iterator[Union[None, bool, int, str, float]]:
 
     def max_elements(self) -> Optional[int]:
         return (
-            self.cdata_leaflist.max
-            if self.cdata_leaflist.max != (2**32 - 1)
-            else None
+            self.cdata_leaflist.max if self.cdata_leaflist.max != (2**32 - 1) else None
         )
 
     def min_elements(self) -> int:

libyang/util.py

@@ -121,4 +121,6 @@ def data_load(in_type, in_data, data, data_keepalive, encode=True):
         c_str = str2c(in_data, encode=encode)
         data_keepalive.append(c_str)
         ret = lib.ly_in_new_memory(c_str, data)
+    else:
+        raise ValueError("invalid input")
     return ret

libyang/xpath.py

@@ -85,7 +85,7 @@ def xpath_split(xpath: str) -> Iterator[Tuple[str, str, List[Tuple[str, str]]]]:
 
 # -------------------------------------------------------------------------------------
 def _xpath_keys_to_key_name(
-    keys: List[Tuple[str, str]]
+    keys: List[Tuple[str, str]],
 ) -> Optional[Union[str, Tuple[str, ...]]]:
     """
     Extract key name from parsed xpath keys returned by xpath_split. The return value

pylintrc

@@ -74,6 +74,7 @@ disable=
 	too-many-branches,
 	too-many-lines,
 	too-many-locals,
+	too-many-positional-arguments,
 	too-many-return-statements,
 	too-many-statements,
 	unused-argument,

tox.ini

@@ -36,8 +36,8 @@ basepython = python3
 description = Format python code using isort and black.
 changedir = .
 deps =
-	black~=23.12.1
-	isort~=5.13.2
+	black~=25.1.0
+	isort~=6.0.0
 skip_install = true
 install_command = python3 -m pip install {opts} {packages}
 allowlist_externals =
@@ -52,14 +52,14 @@ basepython = python3
 description = Run coding style checks.
 changedir = .
 deps =
-	astroid~=3.0.2
-	black~=23.12.1
-	flake8~=7.0.0
-	isort~=5.13.2
-	pycodestyle~=2.11.1
+	astroid~=3.3.8
+	black~=25.1.0
+	flake8~=7.1.1
+	isort~=6.0.0
+	pycodestyle~=2.12.1
 	pyflakes~=3.2.0
-	pylint~=3.0.3
-	setuptools~=69.0.3
+	pylint~=3.3.4
+	setuptools~=75.8.0
 allowlist_externals =
 	/bin/sh
 	/usr/bin/sh