Skip to content

Conversation

@GauBen
Copy link
Member

@GauBen GauBen commented Dec 17, 2025

Description

We might be missing permissions on the on-merge.yml workflow, added the permissions listed here: https://github.com/dorny/test-reporter#example

Checklist

Source code

  • I've shared and documented any breaking change
  • I've reviewed and updated the jahia-depends

Tests

  • I've provided Unit and/or Integration Tests
  • I've updated the parent issue with required manual validations

Tip

Documentation to guide the reviews: How to do a code review

@GauBen GauBen requested a review from a team as a code owner December 17, 2025 08:40
Copilot AI review requested due to automatic review settings December 17, 2025 08:40
Copy link
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 adjusts GitHub Actions workflow permissions for the on-merge.yml workflow by adding the necessary permissions for the test-reporter action. The changes align with the recommended permissions from the dorny/test-reporter documentation.

  • Adds actions: read, checks: write, and contents: read permissions to enable proper test reporting functionality

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@GauBen GauBen enabled auto-merge (squash) December 17, 2025 08:44
@baptistegrimaud
Copy link
Contributor

@GauBen GauBen merged commit 0fbddd5 into main Dec 17, 2025
17 checks passed
@GauBen GauBen deleted the fix/ci-permissions branch December 17, 2025 09:09
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