mirror of
https://github.com/ArduPilot/ardupilot.git
synced 2026-10-13 02:54:09 +08:00
Christian Clauss
123c2efc44
```diff - parts = re.split(r'[,=\s]+', processed_line, 1) + parts = re.split(r'[,=\s]+', processed_line, maxsplit=1) ``` Pytest: `DeprecationWarning: 'maxsplit' is passed as positional argument` * https://docs.python.org/3/library/re.html#re.split ### How was this tested? Localhost testing... % `python -m pytest --ignore=Tools/ros2` ``` ========================================== test session starts ========================================== platform darwin -- Python 3.13.5, pytest-8.4.1, pluggy-1.6.0 rootdir: /Users/cclauss/Python/ardupilot configfile: pyproject.toml collected 53 items Tools/autotest/unittest/annotate_params_test.py ........................ [ 45%] Tools/autotest/unittest/extract_param_defaults_test.py .................. [ 79%] Tools/scripts/param_check_test.py ........... [100%] ========================================== warnings summary ========================================== Tools/scripts/param_check_test.py: 118 warnings /Users/cclauss/Python/ardupilot/Tools/scripts/param_check.py:338: DeprecationWarning: 'maxsplit' is passed as positional argument parts = re.split(r'[,=\s]+', processed_line, 1) -- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html ========================================== 53 passed, 118 warnings in 2.74s ========================================== ``` After this change: % `python -m pytest --ignore=Tools/ros2` ``` ========================================== test session starts ========================================== platform darwin -- Python 3.13.5, pytest-8.4.1, pluggy-1.6.0 rootdir: /Users/cclauss/Python/ardupilot configfile: pyproject.toml collected 53 items Tools/autotest/unittest/annotate_params_test.py ........................ [ 45%] Tools/autotest/unittest/extract_param_defaults_test.py .................. [ 79%] Tools/scripts/param_check_test.py ........... [100%] ========================================== 53 passed in 2.55s ========================================== ``` Also failed with `ruff check --select=B034` and passed after the proposed change. https://docs.astral.sh/ruff/rules/re-sub-positional-args