Skip to content

ci: fix branch-protection-info workflow permissions - #2

Merged
tlcdv merged 1 commit into
mainfrom
ci/fix-branch-protection-permissions
May 30, 2026
Merged

ci: fix branch-protection-info workflow permissions#2
tlcdv merged 1 commit into
mainfrom
ci/fix-branch-protection-permissions

Conversation

@tlcdv

@tlcdv tlcdv commented May 30, 2026

Copy link
Copy Markdown
Contributor

The Branch Protection Info workflow failed with Resource not accessible by integration because its GITHUB_TOKEN was read-only and could not create a PR comment.

  • Add a permissions block granting issues: write and pull-requests: write (PR comments use the issues API), with contents: read.
  • Await the createComment call so any failure is handled instead of surfacing as an unhandled promise rejection.

No application code touched.

@github-actions

Copy link
Copy Markdown

🛡️ Branch Protection Active

This repository uses branch protection rules for the main branch. This means:

What this enables:

  • Quality control through required reviews
  • Prevention of accidental direct pushes to main
  • Safer collaboration for all contributors

📋 For contributors:

  • Always create feature branches for your changes
  • Submit pull requests for review before merging
  • Follow the contributing guidelines in docs/instructions.md

💬 Questions? Please use GitHub Discussions for questions about contributing or the development process.

Thank you for contributing to brain_emulation! 🧠

@tlcdv
tlcdv merged commit f6d4bdb into main May 30, 2026
6 checks passed
@tlcdv
tlcdv deleted the ci/fix-branch-protection-permissions branch May 30, 2026 13:27
tlcdv pushed a commit that referenced this pull request May 30, 2026
…b5-97c7-38582526c602

[WIP] Revise README.md to improve clarity and link to docs
tlcdv added a commit that referenced this pull request May 30, 2026
…ssions

ci: fix branch-protection-info workflow permissions
tlcdv added a commit that referenced this pull request Jun 25, 2026
…b5-97c7-38582526c602

[WIP] Revise README.md to improve clarity and link to docs
tlcdv added a commit that referenced this pull request Jun 25, 2026
…ssions

ci: fix branch-protection-info workflow permissions
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