Skip to content

Commit 210aa55

Browse files
authored
Merge pull request #133 from PecanProject/bump-deploy
bump deploy action version; ~~remove unused~~__add note to__ build.yml
2 parents 40505b1 + a9262c4 commit 210aa55

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

.github/workflows/build.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
# Q: "Why do we need a build job when we already have a build-and-deploy job?"
2+
# A: This one runs to test PRs; build-and-deploy runs to deploy the site after merging.
13
name: build
24
on:
35
pull_request:

.github/workflows/deploy.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ jobs:
1818
npm run build
1919
2020
- name: Deploy 🚀
21-
uses: JamesIves/github-pages-deploy-action@4.1.4
21+
uses: JamesIves/github-pages-deploy-action@v4.8.0
2222
with:
2323
branch: gh-pages # The branch the action should deploy to.
2424
folder: build # The folder the action should deploy.

0 commit comments

Comments
 (0)