Skip to content

Commit 8782b9a

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 9c9ac21 commit 8782b9a

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
@@ -253,13 +253,13 @@ jobs:
253253
254254
${{ steps.coverage-matrix-summary.outputs.table }}
255255
256-
[Full dashboard](https://forgerock.github.io/ping-javascript-sdk/coverage-matrix/)
256+
[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/)
257257
258-
- name: Deploy coverage dashboard to GitHub Pages
258+
- name: Preview coverage dashboard
259259
continue-on-error: true
260260
uses: peaceiris/actions-gh-pages@v4
261261
with:
262262
github_token: ${{ secrets.GITHUB_TOKEN }}
263263
publish_dir: ./tools/coverage-matrix/dist
264-
destination_dir: coverage-matrix
264+
destination_dir: pr-${{ github.event.number }}/${{ github.sha }}/coverage-matrix
265265
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)