Skip to content

Code coverage#110

Open
SimonZanta wants to merge 7 commits into
comgate-payments:masterfrom
SimonZanta:code-coverage
Open

Code coverage#110
SimonZanta wants to merge 7 commits into
comgate-payments:masterfrom
SimonZanta:code-coverage

Conversation

@SimonZanta
Copy link
Copy Markdown
Collaborator

No description provided.

SimonZanta and others added 7 commits May 18, 2026 10:48
- Add coverage-badge.yml workflow triggered on push to master
- Runs Codeception with --coverage --coverage-xml
- Parses coverage % from Clover XML via PHP one-liner
- Updates GitHub Gist via schneegans/dynamic-badges-action
- Adds shields.io coverage badge to README.md (placeholder {GH_USER}/{GIST_ID})

Manual setup required:
- Create GitHub Gist with coverage.json file
- Create PAT with gist scope
- Add GIST_TOKEN and COVERAGE_GIST_ID secrets to repo

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add permissions: pages write + id-token write
- Add --coverage-html to codecept run
- Upload tests/_output/coverage/ as Pages artifact
- Add deploy job using actions/deploy-pages@v4

Manual setup: Repo → Settings → Pages → Source: GitHub Actions
URL: https://comgate-payments.github.io/sdk-php/

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Remove coverage-badge.yml (Gist + GitHub Pages approach)
- Add coverage.yml: runs on push to master, uploads clover XML to Coveralls
- Update README badge to Coveralls

Manual setup: https://coveralls.io → enable repo → add COVERALLS_REPO_TOKEN secret

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Coverage on push to master is now handled by tests.yml directly.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add push: branches: master trigger for baseline coverage update
- Checkout fallback for PR vs push events
- Add coverallsapp/github-action@v2 step for coverage upload

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

1 participant