From 8fe5d437c9fc6f4e766d59cbea7b45ec2d7b5efb Mon Sep 17 00:00:00 2001
From: Daniel Gafni <danielgafni16@gmail.com>
Date: Wed, 1 Nov 2023 22:21:49 +0100
Subject: [PATCH] :construction_worker: fix matrix packages installation in CI
 (#45)

* :bug: fix matrix packages intallation in CI

* re-introduce dagster 1.5 to CI tests
---
 .github/workflows/ci.yml | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 73a4ce8..584a132 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -32,7 +32,7 @@ jobs:
           - "3.8"
         dagster_version:
           - "1.4.0"
-            # - "1.5.0" disabled until https://github.com/dagster-io/dagster/issues/17436 is fixed
+          - "1.5.0"
         polars_version:
           - "0.17.0"
           - "0.18.0"
@@ -51,7 +51,9 @@ jobs:
           virtualenvs-in-project: false
           installer-parallel: true
       - name: Install dependencies
-        run: poetry install --all-extras --sync && pip install --force-reinstall polars~=${{ matrix.polars_version }} dagster~=${{ matrix.dagster_version }}
+        run: poetry install --all-extras --sync
+      - name: Install matrix overrides
+        run: poetry add polars=~${{ matrix.polars_version }} dagster=~${{ matrix.dagster_version }}&& poetry install --sync --all-extras
       - name: Print polars info
         run: python -c 'import polars; print(polars.show_versions())'
       - name: Print dagster info
@@ -74,7 +76,7 @@ jobs:
           - "3.8"
         dagster_version:
           - "1.4.0"
-            # - "1.5.0" disabled until https://github.com/dagster-io/dagster/issues/17436 is fixed
+          - "1.5.0"
         polars_version:
           - "0.17.0"
           - "0.18.0"
@@ -93,7 +95,9 @@ jobs:
           virtualenvs-in-project: false
           installer-parallel: true
       - name: Install dependencies
-        run: poetry install --all-extras --sync && pip install --force-reinstall polars~=${{ matrix.polars_version }} dagster~=${{ matrix.dagster_version }}
+        run: poetry install --all-extras --sync
+      - name: Install matrix overrides
+        run: poetry add polars=~${{ matrix.polars_version }} dagster=~${{ matrix.dagster_version }} && poetry install --sync --all-extras
       - name: Run pre-commit hooks
         run: pre-commit run --all-files