Skip to content

Commit

Permalink
refactor: Simplify deploying to gh pages (#682)
Browse files Browse the repository at this point in the history
* remove test-page/dist to simplify gh pages deploy

* update package.json to use gh-pages and gh-page yml to use
  • Loading branch information
sookburt committed Oct 25, 2022
1 parent 8351b81 commit ac62022
Show file tree
Hide file tree
Showing 7 changed files with 146 additions and 2,715 deletions.
8 changes: 5 additions & 3 deletions .github/workflows/gh-pages.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,9 @@ jobs:
run: yarn build

- name: Deploy
run: |
git remote set-url origin https://git:${GITHUB_TOKEN}@github.com/${GITHUB_REPOSITORY}.git
yarn github-pages-deploy
env:
GH_PAT: ${{ secrets.GH_PAT }}
BUILD_DIR: 'test-page/dist/'
uses: guardian/ghpages@v0.4.0
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ dist
.gh-pages
/coverage
.idea/
test-page/dist

# Cypress Tests
cypress/dist
Expand Down
2 changes: 2 additions & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
"compile": "rollup --config",
"dev": "yarn clean && yarn compile --watch",
"fix": "yarn lint --fix",
"github-pages-deploy": "gh-pages -d test-page/dist -u \"github-actions-bot <support+actions@github.com>\"",
"lint": "eslint . --ext=js,ts,tsx",
"prepublishOnly": "yarn build",
"readme-toc:commit": "git add README.md && git commit -m 'Update README toc' || true",
Expand Down Expand Up @@ -72,6 +73,7 @@
"eslint-plugin-jest": "^26",
"eslint-plugin-prettier": "^3",
"eslint-plugin-svelte3": "^3",
"gh-pages": "^4.0.0",
"husky": "^7.0.0",
"jest": "^26",
"locate-character": "^2",
Expand Down
3 changes: 0 additions & 3 deletions test-page/dist/bundle.css

This file was deleted.

12 changes: 0 additions & 12 deletions test-page/dist/index.html

This file was deleted.

Loading

0 comments on commit ac62022

Please sign in to comment.