Propagate lkg flag to sub-workflows

Signed-off-by: Keith Battocchi <kebatt@microsoft.com>
py-why · Jun 28, 2023 · a35ba78 · a35ba78
1 parent 7a192df
commit a35ba78
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 7 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -13,7 +13,7 @@ on:
         type: string
       use_lkg:
         description: 'Whether to use the last known good versions of dependencies'
-        required: true
+        required: false
         default: True
         type: boolean
   # nightly
@@ -28,7 +28,7 @@ concurrency:
 # Precompute the ref if the workflow was triggered by a workflow dispatch rather than copying this logic repeatedly
 env:
   ref: ${{ github.event_name == 'workflow_dispatch' && inputs.ref || null }}
-  use_lkg: ${{ (github.event_name == 'workflow_dispatch' && inputs.use_lkg || github.event_name == 'pull_request') || null }}
+  use_lkg: ${{ (github.event_name == 'workflow_dispatch' && inputs.use_lkg) || github.event_name == 'pull_request' }}
 
 jobs:
   eval: 
@@ -288,6 +288,7 @@ jobs:
       repository: testpypi
       # don't have access to env context here for some reason
       ref: ${{ github.event_name == 'workflow_dispatch' && inputs.ref || null }}
+      use_lkg: ${{ (github.event_name == 'workflow_dispatch' && inputs.use_lkg) || github.event_name == 'pull_request' }}
 
   docs:
     name: Build documentation
@@ -299,7 +300,8 @@ jobs:
       environment: test
       # don't have access to env context here for some reason
       ref: ${{ github.event_name == 'workflow_dispatch' && inputs.ref || null }}
-
+      use_lkg: ${{ (github.event_name == 'workflow_dispatch' && inputs.use_lkg) || github.event_name == 'pull_request' }}
+
   verify:
     name: Verify CI checks
     needs: [lint, notebooks, tests, build, docs]

diff --git a/.github/workflows/publish-documentation.yml b/.github/workflows/publish-documentation.yml
@@ -18,6 +18,11 @@ on:
         required: false
         default: ''
         type: string
+      use_lkg:
+        description: 'Whether to use the last known good versions of dependencies'
+        required: false
+        default: True
+        type: boolean
   # annoyingly, there does not seem to be a way to share these input definitions between triggers
   workflow_call:
     inputs:
@@ -37,7 +42,11 @@ on:
         required: false
         default: ''
         type: string
-
+      use_lkg:
+        description: 'Whether to use the last known good versions of dependencies'
+        required: false
+        default: True
+        type: boolean
 
 jobs:
   create_docs:
@@ -54,7 +63,7 @@ jobs:
           python-version: 3.8 # because of our supported TensorFlow versions, must build on 3.6-3.8
       - run: python -m pip install --upgrade pip && pip install --upgrade setuptools
         name: Ensure latest pip and setuptools
-      - run: pip install -U cython && pip install -e .[all]
+      - run: pip install -U cython && pip install -e .[all] ${{ inputs.use_lkg && '-r lkg.txt' }}
         name: Install econml[all]
       - run: sudo apt-get -yq install graphviz
         name: Install graphviz

diff --git a/.github/workflows/publish-package.yml b/.github/workflows/publish-package.yml
@@ -19,6 +19,12 @@ on:
         required: false
         default: ''
         type: string
+      use_lkg:
+          description: 'Whether to use the last known good versions of dependencies'
+          required: false
+          default: True
+          type: boolean
+
   # annoyingly, there does not seem to be a way to share these input definitions between triggers
   workflow_call:
     inputs:
@@ -38,7 +44,11 @@ on:
         required: false
         default: ''
         type: string
-
+      use_lkg:
+        description: 'Whether to use the last known good versions of dependencies'
+        required: false
+        default: True
+        type: boolean
 jobs:
   build_wheels:
     name: Build wheels on ${{ matrix.os }}
@@ -81,7 +91,7 @@ jobs:
           python-version: 3.8 # because of our supported TensorFlow versions, must build on 3.6-3.8
       - run: python -m pip install --upgrade pip && pip install --upgrade setuptools
         name: Ensure latest pip and setuptools
-      - run: pip install -U cython && pip install -e .[all]
+      - run: pip install -U cython && pip install -e .[all] ${{ inputs.use_lkg && '-r lkg.txt' }}
         name: Install econml[all]
       - run: python setup.py sdist
         name: Build sdist