Skip to content

Commit

Permalink
Merge pull request #59 from neuromatch/release-v0.1.1
Browse files Browse the repository at this point in the history
Fix bugs from the first draft
  • Loading branch information
iamzoltan authored Jun 13, 2024
2 parents 27ebcbc + 71a7d9c commit 8915c97
Show file tree
Hide file tree
Showing 3 changed files with 21 additions and 2 deletions.
15 changes: 14 additions & 1 deletion .github/actions/check-notebooks/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,13 +43,26 @@ runs:
# with:
# output: " "

- name: Get changed files
if: "!contains(env.COMMIT_MESSAGE, 'skip ci')"
id: changed-files
uses: tj-actions/changed-files@v35

- name: List all changed files
run: |
for file in ${{ steps.changed-files.outputs.all_changed_files }}; do
echo "$file was changed."
done
shell: bash -l {0}

- name: Get notebooks
run: |
if ${{ inputs.run_all == 'true' }}; then
# nbs="tutorials/*/*Tutorial*.ipynb";
nbs="tutorials/W2D5_*/*Tutorial*.ipynb";
else
nbs=`python ci/select_notebooks.py ${{ steps.changes.outputs.files }}`;
nbs=`python ci/select_notebooks.py ${{ steps.changed-files.outputs.all_changed_files }}`;
# nbs=`python ci/select_notebooks.py ${{ steps.changes.outputs.files }}`;
fi
echo "NBS=$nbs" >> $GITHUB_ENV
shell: bash -l {0}
Expand Down
6 changes: 6 additions & 0 deletions .github/actions/setup/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,12 @@ inputs:
runs:
using: composite
steps:
- name: Checkout
uses: actions/checkout@v3
with:
persist-credentials: false
fetch-depth: 0
ref: ${{ github.head_ref }}
# - name: Set up Python (conda)
# uses: conda-incubator/setup-miniconda@v3
# with:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/notebook-pr.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ jobs:
- name: Run Notebooks
uses: ./.github/actions/check-notebooks
with:
run_all: 'true'
run_all: 'false'
# exec_flag: '--check-only'
exec_flag: '--execute'
# exercise-continue-on-error: true
Expand Down

0 comments on commit 8915c97

Please sign in to comment.