Refs #20965: Add some comments

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
eProsima · May 9, 2024 · a41da1f · a41da1f
1 parent f63d0ad
commit a41da1f
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 0 deletions.
diff --git a/.github/workflows/reusable-ubuntu-ci.yml b/.github/workflows/reusable-ubuntu-ci.yml
@@ -119,6 +119,8 @@ jobs:
           ref: ${{ steps.get_shapes_demo_branch.outputs.deduced_branch }}
 
       # Required for Shapes Demo
+      # Do not setup python as it will internally modify the pythonLocation env variable
+      # and we want to use a fix version
       - name: Install Qt
         uses: jurplel/install-qt-action@v2.13.0
         with:

diff --git a/tools/fastdds/fastdds.bat b/tools/fastdds/fastdds.bat
@@ -21,11 +21,13 @@ if not %ERRORLEVEL%==0 (
    )
 )
 
+:: Python version in the form "Major.Minor"
 if %PYTHON_VERSION%=="" (
       echo error retrieving python version. Please, make sure python is installed and accessible.
       exit /B 65
    )
 )
 
 :: Use launcher to profit from shebang hints on fastdds.py
+:: Select the correct python version to source the appropriate paths
 py -%PYTHON_VERSION% "%dir%\..\tools\fastdds\fastdds.py" %args%
diff --git a/tools/fastdds/ros-discovery.bat b/tools/fastdds/ros-discovery.bat
@@ -21,11 +21,13 @@ if not %ERRORLEVEL%==0 (
    )
 )
 
+:: Python version in the form "Major.Minor"
 if %PYTHON_VERSION%=="" (
       echo error retrieving python version. Please, make sure python is installed and accessible.
       exit /B 65
    )
 )
 
 :: Use launcher to profit from shebang hints on fastdds.py
+:: Select the correct python version to source the appropriate paths
 py -%PYTHON_VERSION% "%dir%\..\tools\fastdds\fastdds.py" discovery %args%