Skip to content

ci: fix pull-requests permission for jacoco-report coverage comments#207

Merged
blindzero merged 2 commits intomainfrom
copilot/fix-ci-pipeline-error
Feb 22, 2026
Merged

ci: fix pull-requests permission for jacoco-report coverage comments#207
blindzero merged 2 commits intomainfrom
copilot/fix-ci-pipeline-error

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 22, 2026

The Pester (ubuntu-latest) job was failing with HttpError: Resource not accessible by integration because madrapps/jacoco-report posts coverage comments via the Issues Comments API, which requires pull-requests: write — a permission missing from the test job.

Change

  • Added pull-requests: write to the test job's permissions block in ci.yml
permissions:
  contents: read
  actions: write
  checks: write
  pull-requests: write  # required by madrapps/jacoco-report to post PR comments

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI self-assigned this Feb 22, 2026
…rror

Co-authored-by: blindzero <13959569+blindzero@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix CI pipeline error with Pester ci: fix pull-requests permission for jacoco-report coverage comments Feb 22, 2026
Copilot AI requested a review from blindzero February 22, 2026 00:22
@blindzero blindzero marked this pull request as ready for review February 22, 2026 00:26
Copilot AI review requested due to automatic review settings February 22, 2026 00:26
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a CI workflow permission issue where the Pester test job was failing with HttpError: Resource not accessible by integration when the madrapps/jacoco-report action attempted to post coverage comments to pull requests.

Changes:

  • Added pull-requests: write permission to the test job in the CI workflow to enable the jacoco-report action to post PR comments via the GitHub Issues API

@github-actions
Copy link
Copy Markdown

Code Coverage Report

Overall Project 72.19% 🍏

There is no coverage information present for the Files changed

@blindzero blindzero merged commit 8a08b2c into main Feb 22, 2026
9 checks passed
@blindzero blindzero deleted the copilot/fix-ci-pipeline-error branch February 22, 2026 13:21
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.

3 participants