Skip to content

Commit 60493b7

Browse files
committed
fix: deploy PR dashboard to preview path, use relative base
PR deploys to pr-{number}/{sha}/coverage-matrix/ alongside docs preview. Main deploys to /coverage-matrix/ from publish.yml. Changed Vite base to ./ so the same build works at any URL depth.
1 parent 399e914 commit 60493b7

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

.github/workflows/ci.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -189,13 +189,13 @@ jobs:
189189
190190
${{ steps.coverage-matrix-summary.outputs.table }}
191191
192-
[Full dashboard](https://forgerock.github.io/ping-javascript-sdk/coverage-matrix/)
192+
[Full dashboard](https://forgerock.github.io/ping-javascript-sdk/coverage-matrix/) · [PR preview](https://ForgeRock.github.io/ping-javascript-sdk/pr-${{ github.event.number }}/${{ github.sha }}/coverage-matrix/)
193193
194-
- name: Deploy coverage dashboard to GitHub Pages
194+
- name: Preview coverage dashboard
195195
continue-on-error: true
196196
uses: peaceiris/actions-gh-pages@v4
197197
with:
198198
github_token: ${{ secrets.GITHUB_TOKEN }}
199199
publish_dir: ./tools/coverage-matrix/dist
200-
destination_dir: coverage-matrix
200+
destination_dir: pr-${{ github.event.number }}/${{ github.sha }}/coverage-matrix
201201
keep_files: true

tools/coverage-matrix/vite.config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { svelte } from '@sveltejs/vite-plugin-svelte';
33

44
export default defineConfig({
55
root: __dirname,
6-
base: '/ping-javascript-sdk/coverage-matrix/',
6+
base: './',
77
plugins: [svelte()],
88
build: {
99
outDir: './dist',

0 commit comments

Comments
 (0)