Add pytables to test_requirements remove from workflow

Signed-off-by: Ankita Katiyar <ankitakatiyar2401@gmail.com>
tingtingQB · Apr 19, 2023 · 8203daa · 8203daa
1 parent 6552c0d
commit 8203daa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 5 deletions.
diff --git a/.github/workflows/check-plugin.yml b/.github/workflows/check-plugin.yml
@@ -43,11 +43,7 @@ jobs:
         run: |
           cd ${{ inputs.plugin }}
           pip install -r test_requirements.txt
-      - name: Install pytables (only for kedro-datasets on windows)
-        if: matrix.os == 'windows-latest' && inputs.plugin == 'kedro-datasets'
-        run: pip install tables
-      - name: pip freeze
-        run: pip freeze
+          pip freeze
       - name: Run unit tests for Linux / all plugins
         if: matrix.os != 'windows-latest'
         run: make plugin=${{ inputs.plugin }} test

diff --git a/kedro-datasets/test_requirements.txt b/kedro-datasets/test_requirements.txt
@@ -55,6 +55,7 @@ snowflake-snowpark-python~=1.0.0; python_version == '3.8'
 SQLAlchemy>=1.4, <3.0  # The `Inspector.has_table()` method replaces the `Engine.has_table()` method in version 1.4.
 tables~=3.6.0; platform_system == "Windows" and python_version < '3.9'
 tables~=3.6; platform_system != "Windows"
+tables~=3.8.0; platform_system == "Windows" and python_version == '3.10'
 tensorflow-macos~=2.0; platform_system == "Darwin" and platform_machine == "arm64"
 tensorflow~=2.0; platform_system != "Darwin" or platform_machine != "arm64"
 triad>=0.6.7, <1.0