Skip to content

Conversation

@aadithya2112
Copy link
Contributor

@aadithya2112 aadithya2112 commented Jan 3, 2026

Fixes workflow error in pull-request-target.yml

Error: https://github.com/PalisadoesFoundation/.github/actions/runs/20677555661

Cause: GITHUB_TOKEN cannot be declared in workflow_call secrets (it's auto-available).

Fix: Removed lines 5-7. Token still works on line 58.

Summary by CodeRabbit

  • Chores
    • Updated CI/CD workflow configuration to streamline secret handling processes.

✏️ Tip: You can customize this high-level summary in your review settings.

GITHUB_TOKEN is auto-available and can't be declared as a secret.
Fixes: 'secret name GITHUB_TOKEN collides with system reserved name'
@coderabbitai
Copy link

coderabbitai bot commented Jan 3, 2026

Walkthrough

A GITHUB_TOKEN secret requirement was removed from the pull-request-target workflow configuration, eliminating explicit secret validation for workflow_dispatch triggers while preserving all existing workflow triggers and job steps.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
\.github/workflows/pull-request-target\.yml
Removed GITHUB_TOKEN from workflow_call.secrets specification, eliminating explicit secret validation requirement

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A secret's released from its careful bindings,
The workflow flows lighter with simpler findings,
One less requirement, one less to declare—
GitHub Actions hops freely through open air! ✨

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: removing GITHUB_TOKEN from workflow_call secrets in the PR Target Workflow file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 330bc24 and f5ef38a.

📒 Files selected for processing (1)
  • .github/workflows/pull-request-target.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/pull-request-target.yml

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@palisadoes palisadoes merged commit e4c3db7 into PalisadoesFoundation:main Jan 3, 2026
3 checks passed
@palisadoes
Copy link
Contributor

  1. The pull-request target job didn't run for this
  2. It should have automatically closed the PR because there is no linked issue
  3. Please investigate and test

@palisadoes
Copy link
Contributor

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