From 810bb77f7c84fa91a30fa484a3746d11c0c280b2 Mon Sep 17 00:00:00 2001 From: mosfet80 <10235105+mosfet80@users.noreply.github.com> Date: Wed, 7 Jan 2026 15:09:13 +0100 Subject: [PATCH 1/4] fix setuptools deprecations fix UserWarning: Unknown distribution option: 'tests_require' --- setup.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/setup.py b/setup.py index 23ae6b9..8c2545c 100644 --- a/setup.py +++ b/setup.py @@ -23,7 +23,11 @@ maintainer_email="jafar@picknik.ai", description="Python library for loading parameters in launch files", license="BSD", - tests_require=["pytest"], + extras_require={ + 'test': [ + 'pytest', + ], + }, entry_points={ "console_scripts": [], }, From 58aff1f23ab8caf7e39028ba435d3d21d2d91888 Mon Sep 17 00:00:00 2001 From: mosfet80 <10235105+mosfet80@users.noreply.github.com> Date: Mon, 4 May 2026 17:16:20 +0200 Subject: [PATCH 2/4] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- setup.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/setup.py b/setup.py index 8c2545c..770b545 100644 --- a/setup.py +++ b/setup.py @@ -24,8 +24,8 @@ description="Python library for loading parameters in launch files", license="BSD", extras_require={ - 'test': [ - 'pytest', + "test": [ + "pytest", ], }, entry_points={ From 5fa79f43cee9b01c50eea18469e8495ffb2d44eb Mon Sep 17 00:00:00 2001 From: mosfet80 <10235105+mosfet80@users.noreply.github.com> Date: Mon, 4 May 2026 17:21:29 +0200 Subject: [PATCH 3/4] Update GitHub Actions to use latest versions --- .github/workflows/ci.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 6ae2ffe..ff94b32 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -29,12 +29,12 @@ jobs: name: ${{ matrix.env.ROS_DISTRO }}-${{ matrix.env.ROS_REPO }} runs-on: ubuntu-latest steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v6 - name: industrial_ci uses: 'ros-industrial/industrial_ci@master' env: ${{ matrix.env }} - name: upload test artifacts (on failure) - uses: actions/upload-artifact@v2 + uses: actions/upload-artifact@v7 if: failure() with: name: test-results From 2f2a0aa4967c8c27689451ed736dd7089eb2ef1c Mon Sep 17 00:00:00 2001 From: mosfet80 <10235105+mosfet80@users.noreply.github.com> Date: Tue, 12 May 2026 22:31:27 +0200 Subject: [PATCH 4/4] Refactor exception handling in file reading functions Refactor exception handling for file operations to improve readability. --- launch_param_builder/utils.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/launch_param_builder/utils.py b/launch_param_builder/utils.py index b2a517e..933ddd0 100644 --- a/launch_param_builder/utils.py +++ b/launch_param_builder/utils.py @@ -62,7 +62,9 @@ def load_file(file_path: Path): try: with open(file_path, "r") as file: return file.read() - except EnvironmentError: # parent of IOError, OSError *and* WindowsError where available + except ( + EnvironmentError + ): # parent of IOError, OSError *and* WindowsError where availabl return None @@ -72,7 +74,9 @@ def load_yaml(file_path: Path): try: with open(file_path, "r") as file: return yaml.load(file, Loader=yaml.FullLoader) - except EnvironmentError: # parent of IOError, OSError *and* WindowsError where available + except ( + EnvironmentError + ): # parent of IOError, OSError *and* WindowsError where availabl return None