Skip to content

chore: fix coverage report publishing to GH pages - #285

Merged
mcnulty-fp merged 1 commit into
mainfrom
chore/INTER-2599-fix-coverage-report
Sep 28, 2026
Merged

mcnulty-fp merged 1 commit into
mainfrom
chore/INTER-2599-fix-coverage-report

Conversation

@mcnulty-fp

@mcnulty-fp mcnulty-fp commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Now that signed commits are required, JamesIves/github-pages-deploy-action can no longer push to the gh-pages branch. Followed the same solution from this PR: fingerprintjs/python-sdk#228

Changes:

  • Replace the gh-pages branch push with actions/upload-pages-artifact + actions/deploy-pages
  • Add deploy-pages job to complete GH pages deployment
  • Drop the workflow-level contents: write permission, since nothing pushes to a branch anymore

⚠️ Before merging

Switch the Pages source to GitHub Actions. More details here. Requires repo admin, so someone with that access will need to run this (from the UI, or via the gh API command below):

gh api -X PUT repos/fingerprintjs/php-sdk/pages -f build_type=workflow

Slack ref relevant to this process: https://fingerprintjs.slack.com/archives/C050TKU9X5L/p1788168387519619

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ This PR doesn't contain any changesets. If there are user-facing changes, don't forget to run:

pnpm exec changeset

to create a changeset.

@github-actions

Copy link
Copy Markdown
Contributor

🟢 Coverage increased by 97.52%

Code Coverage Report

Coverage Report

St. Category Percentage Covered / Total
🟢 Statements 97.52% 3655 / 3748
🟢 Functions 98.45% 1464 / 1487
Files Coverage
St. File Methods Statements Total Coverage

@mcnulty-fp
mcnulty-fp marked this pull request as ready for review September 28, 2026 15:19
@mcnulty-fp
mcnulty-fp merged commit a4fc366 into main Sep 28, 2026
18 checks passed
@mcnulty-fp
mcnulty-fp deleted the chore/INTER-2599-fix-coverage-report branch September 28, 2026 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants