Skip to content

Commit

Permalink
Merge pull request #89 from leap-stc/remove-local-copies-of-pythia-wo…
Browse files Browse the repository at this point in the history
…rkflows

Fix website publishing workflow
  • Loading branch information
jbusecke authored Apr 16, 2024
2 parents bdc4a1a + 63a494f commit 956e5a3
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 301 deletions.
207 changes: 0 additions & 207 deletions .github/workflows/build-book.yaml

This file was deleted.

85 changes: 0 additions & 85 deletions .github/workflows/deploy-book.yaml

This file was deleted.

9 changes: 1 addition & 8 deletions .github/workflows/publish-website.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,7 @@ permissions:

jobs:
build:
# uses: ProjectPythia/cookbook-actions/.github/workflows/build-book.yaml@main
# until https://github.com/ProjectPythia/cookbook-actions/pull/65 is merged
# FIXME: I keep running into this annoying issue that I cannot use a PR to an existing public workflow. Very frustrating
# uses: jbusecke/cookbook-actions/.github/workflows/build-book.yaml@patch-4
# So the only way to fix this rn is to manually copy the PR content here and reference it locally. Silly.
uses: ./.github/workflows/build-book.yaml
uses: ProjectPythia/cookbook-actions/.github/workflows/build-book.yaml@main
with:
environment_file: 'website/environment.yml'
environment_name: climsim-docs-env
Expand All @@ -27,8 +22,6 @@ jobs:
output_path: '_build/html'
deploy:
needs: build
#Trying this out to see if the auth problem is fixed
# uses: ./.github/workflows/deploy-book.yaml #hack until https://github.com/ProjectPythia/cookbook-actions/pull/45 is merged.
uses: ProjectPythia/cookbook-actions/.github/workflows/deploy-book.yaml@main
with:
publish_dir: "website/_build/html"
2 changes: 1 addition & 1 deletion website/environment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@ name: climsim-docs-env
channels:
- conda-forge
dependencies:
- jupyter-book
- jupyter-book

0 comments on commit 956e5a3

Please sign in to comment.