Skip to content

docs: Elevate GitHub permissions to all workflow examples#46

Merged
joeltjames merged 1 commit intomainfrom
add-github-permissions-requirement
Feb 25, 2026
Merged

docs: Elevate GitHub permissions to all workflow examples#46
joeltjames merged 1 commit intomainfrom
add-github-permissions-requirement

Conversation

@danielma
Copy link
Member

Add explicit pull-requests and contents permissions to all GitHub Actions workflow examples throughout the documentation.

Problem

The action was failing in downstream projects (e.g., ChurchCenterApp) when trying to post comments to PRs because the necessary GitHub permissions were not explicitly configured in the workflow examples.

See: https://github.com/planningcenter/ChurchCenterApp/actions/runs/21845368960/job/63039860783
Related PR: https://github.com/planningcenter/ChurchCenterApp/pull/4716

Solution

Added the required permissions block to all workflow examples:

permissions:
  pull-requests: write  # For posting comments
  contents: read        # For reading PR data

Changes

  • README.md basic usage
  • Installation guide
  • Your First Rule guide
  • 6 example workflows
  • 4 migration workflows
  • Documentation homepage

These permissions are now prominently featured across all documentation to prevent similar issues in the future.

Add explicit pull-requests and contents permissions to all GitHub Actions
workflow examples throughout the documentation. These permissions are
required for the action to function properly:

- pull-requests: write - Required for posting comments to PRs
- contents: read - Required for reading PR data

This resolves workflow failures observed in ChurchCenterApp (PR #4716)
where the action failed to post comments due to missing permissions.

The permissions are now prominently featured in:
- README.md basic usage example
- Installation guide workflow
- Your First Rule guide
- All example workflows (6 examples)
- Migration guide examples (4 workflows)
- Documentation homepage

Fixes failures like: planningcenter/ChurchCenterApp#4716
See: https://github.com/planningcenter/ChurchCenterApp/actions/runs/21845368960
@danielma
Copy link
Member Author

I did this all with Copilot in a codespace. I don't know why it didn't co-author!

@joeltjames joeltjames merged commit a39b327 into main Feb 25, 2026
1 check passed
@joeltjames joeltjames deleted the add-github-permissions-requirement branch February 25, 2026 23:36
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