Skip to content

fix(ci): Permissions missing for reusable_ci workflow#57

Merged
gxmiranda merged 1 commit into
complytime:mainfrom
gxmiranda:fix/workflow_permission_breaking_ci
Jun 19, 2026
Merged

fix(ci): Permissions missing for reusable_ci workflow#57
gxmiranda merged 1 commit into
complytime:mainfrom
gxmiranda:fix/workflow_permission_breaking_ci

Conversation

@gxmiranda

@gxmiranda gxmiranda commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Add missing pull-requests: read permission to the call_reusable_ci job in the CI workflow. Without this permission, the reusable CI workflow fails when triggered by pull request events.

Related Issues

Closes #58

Review Hints

  • Single line addition in .github/workflows/ci.yml
  • Adds pull-requests: read to the call_reusable_ci job permissions block

@gxmiranda gxmiranda requested a review from a team as a code owner June 19, 2026 12:30
@gxmiranda gxmiranda requested review from em-redhat, gvauter and yvonnedevlinrh and removed request for a team June 19, 2026 12:30
@gxmiranda gxmiranda force-pushed the fix/workflow_permission_breaking_ci branch from 943668d to c2822c2 Compare June 19, 2026 13:00
@gxmiranda gxmiranda changed the title Temporary fix for issue 56: merging PR 52 introduced CI workflow permission issue fix(ci): Permissions missing for reusable_ci workflow Jun 19, 2026
@gxmiranda gxmiranda force-pushed the fix/workflow_permission_breaking_ci branch from c2822c2 to a85c246 Compare June 19, 2026 13:06

@yvonnedevlinrh yvonnedevlinrh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@em-redhat em-redhat left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fixes issue as described. lgtm

@gxmiranda gxmiranda merged commit f4d057f into complytime:main Jun 19, 2026
7 checks passed
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.

bug: add missing pull-requests: read permission for call_reusable_ci job

3 participants