Skip to content

Use binderbot switch #92

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 34 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
34 commits
Select commit Hold shift + click to select a range
4e894b6
First attempt at using new binderbot-enabled workflow
brian-rose Oct 14, 2022
a8f6447
Fix up environments
brian-rose Oct 14, 2022
2271bbf
Tweak to force rerun
brian-rose Oct 14, 2022
8594df9
Trivial change
brian-rose Oct 17, 2022
af2bea9
Use right env for link-checker
brian-rose Oct 17, 2022
190ad49
Default to current repo for binder env
brian-rose Oct 17, 2022
c9a0ab7
Trigger another build
brian-rose Oct 17, 2022
1a30263
Execute all notebooks in repo
brian-rose Oct 17, 2022
eecb192
Trigger rebuild
brian-rose Oct 17, 2022
a78253c
Simpler defaults
brian-rose Oct 17, 2022
08fe670
Consistency across actions
brian-rose Oct 17, 2022
c676a2c
Do not execute notebooks during jupyterbook build
brian-rose Oct 17, 2022
f4edb17
Trigger rebuild
brian-rose Oct 17, 2022
546ff66
Try again
brian-rose Oct 17, 2022
cafef6e
And again
brian-rose Oct 17, 2022
aef6950
Try using switch in config file
brian-rose Nov 5, 2022
bd31672
Put juptyerbook back in environment.yml for now
brian-rose Nov 5, 2022
09a11a6
Fix and run again
brian-rose Nov 5, 2022
5f9bad1
Spell it right
brian-rose Nov 5, 2022
1017298
Trigger rerun
brian-rose Nov 5, 2022
31bd10a
Trigger rebuild
brian-rose Nov 7, 2022
0263ae9
Do it again
brian-rose Nov 7, 2022
26a6b00
And again
brian-rose Nov 7, 2022
e0e984b
Try again again
brian-rose Nov 7, 2022
8d1e11a
Again
brian-rose Nov 7, 2022
b884fca
again
brian-rose Nov 7, 2022
29f238b
Can we use cached environment?
brian-rose Nov 7, 2022
cf9c007
Some cleanup
brian-rose Nov 7, 2022
5fb4123
Now try switching back to github execution
brian-rose Nov 7, 2022
077e7c0
Full env for github-based execution
brian-rose Nov 7, 2022
019442a
Back to binderbot
brian-rose Nov 7, 2022
18c86c8
Should use cache this time
brian-rose Nov 7, 2022
e0a24f2
One more test
brian-rose Nov 7, 2022
b8b8780
Rebuild env
brian-rose Nov 7, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 4 additions & 10 deletions .github/workflows/nightly-build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,16 +8,10 @@ on:
jobs:
build:
if: ${{ github.repository_owner == 'ProjectPythiaCookbooks' }}
uses: ProjectPythiaCookbooks/cookbook-actions/.github/workflows/build-book.yaml@main
with:
environment_name: cookbook-dev
environment_file: environment.yml
path_to_notebooks: ./
# uses: ProjectPythiaCookbooks/cookbook-actions/.github/workflows/build-book.yaml@main
uses: brian-rose/cookbook-actions/.github/workflows/build-book.yaml@binderbot-switch

link-check:
if: ${{ github.repository_owner == 'ProjectPythiaCookbooks' }}
uses: ProjectPythiaCookbooks/cookbook-actions/.github/workflows/link-checker.yaml@main
with:
environment_name: cookbook-dev
environment_file: environment.yml
path_to_notebooks: ./
# uses: ProjectPythiaCookbooks/cookbook-actions/.github/workflows/link-checker.yaml@main
uses: brian-rose/cookbook-actions/.github/workflows/link-checker.yaml@binderbot-switch
4 changes: 0 additions & 4 deletions .github/workflows/publish-book.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,10 +10,6 @@ on:
jobs:
build:
uses: ProjectPythiaCookbooks/cookbook-actions/.github/workflows/build-book.yaml@main
with:
environment_name: cookbook-dev
environment_file: environment.yml
path_to_notebooks: ./

deploy:
needs: build
Expand Down
8 changes: 2 additions & 6 deletions .github/workflows/trigger-book-build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,7 @@ on:

jobs:
build:
uses: ProjectPythiaCookbooks/cookbook-actions/.github/workflows/build-book.yaml@main
# uses: ProjectPythiaCookbooks/cookbook-actions/.github/workflows/build-book.yaml@main
uses: brian-rose/cookbook-actions/.github/workflows/build-book.yaml@binderbot-switch
with:
environment_name: cookbook-dev
environment_file: environment.yml
path_to_notebooks: ./
use_cached_environment: 'true' # This is default, not strickly needed. Set to 'false' to always build a new environment
artifact_name: book-zip-${{ github.event.number }}

10 changes: 3 additions & 7 deletions .github/workflows/trigger-link-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,6 @@ on:

jobs:
link-check:
uses: ProjectPythiaCookbooks/cookbook-actions/.github/workflows/link-checker.yaml@main
with:
environment_name: cookbook-dev
environment_file: environment.yml
path_to_notebooks: ./
use_cached_environment: 'true'

# uses: ProjectPythiaCookbooks/cookbook-actions/.github/workflows/link-checker.yaml@main
uses: brian-rose/cookbook-actions/.github/workflows/link-checker.yaml@binderbot-switch

6 changes: 2 additions & 4 deletions _config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,9 @@ tags:
packages:
- samplepackage

# Execute the notebooks upon build
# Don't execute the notebooks upon build (we do that via binderbot)
execute:
execute_notebooks: cache
timeout: 600
allow_errors: True
execute_notebooks: binder

# Add a few extensions to help with parsing content
parse:
Expand Down
2 changes: 1 addition & 1 deletion environment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,4 @@ dependencies:
- jupyterlab
- pip
- pip:
- sphinx-pythia-theme
- sphinx-pythia-theme