Skip to content

Commit

Permalink
remove debug output and use reusable job
Browse files Browse the repository at this point in the history
  • Loading branch information
Lendemor committed Nov 29, 2024
1 parent c63a002 commit 4662901
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 25 deletions.
24 changes: 7 additions & 17 deletions .github/workflows/check_export.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,24 +13,14 @@ on:
description: 'Reflex dep (raw pip spec)'

jobs:
list-examples:
runs-on: ubuntu-latest
outputs:
examples: ${{ steps.generate-matrix.outputs.examples }}
steps:
- uses: actions/checkout@v4
- name: Generate Matrix
id: generate-matrix
run: |
EXAMPLES="$(find . -not -name '.*' -maxdepth 1 -type d | cut -f2 -d/ | sort | grep -vw chakra_apps | jq -R | jq -s -c)"
echo $EXAMPLES
echo "examples=$EXAMPLES" >> $GITHUB_OUTPUT
list-templates:
uses: ./.github/workflows/list-templates.yml

check-export:
needs: [list-examples]
needs: [list-templates]
strategy:
matrix:
example: ${{ fromJSON(needs.list-examples.outputs.examples) }}
folder: ${{ fromJSON(needs.list-templates.outputs.templates) }}
fail-fast: false
runs-on: ubuntu-latest
steps:
Expand All @@ -42,7 +32,7 @@ jobs:
python-version: 3.9
- id: export-check
run: |
f=${{ matrix.example }}
f=${{ matrix.folder }}
if [[ ! -d $f ]]; then
echo "$f is not a directory!"
exit 1
Expand Down Expand Up @@ -80,7 +70,7 @@ jobs:
- name: Check for DeprecationWarning in logs
run: |
cd ${{ matrix.example }}
cd ${{ matrix.folder }}
dep_lines=$(grep -i "DeprecationWarning:" export_logs.txt || true)
if [ -n "$dep_lines" ]; then
echo "Found Deprecation warning:"
Expand Down
8 changes: 0 additions & 8 deletions .github/workflows/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,14 +13,6 @@ jobs:
list-templates:
uses: ./.github/workflows/list-templates.yml

debug-output:
needs: list-templates
runs-on: ubuntu-latest
steps:
- name: Debug output
run: |
echo "Templates: ${{ needs.list-templates.outputs.templates }}"
deploy:
needs: list-templates
runs-on: ubuntu-latest
Expand Down

0 comments on commit 4662901

Please sign in to comment.