Skip to content

[GH-2964] Remove first-interaction workflow#2965

Merged
jbampton merged 1 commit into
apache:masterfrom
jiayuasu:chore/remove-first-interaction-workflow
May 18, 2026
Merged

[GH-2964] Remove first-interaction workflow#2965
jbampton merged 1 commit into
apache:masterfrom
jiayuasu:chore/remove-first-interaction-workflow

Conversation

@jiayuasu
Copy link
Copy Markdown
Member

Did you read the Contributor Guide?

Is there any related issue?

Closes #2964

What changes were proposed in this PR?

Removes .github/workflows/first-interaction.yml.

Why are the changes needed?

The workflow has been failing on every first-time fork-PR with:

HttpError: Resource not accessible by integration
status: 403
url: https://api.github.com/repos/apache/sedona/issues/<n>/comments

Example failure: https://github.com/apache/sedona/actions/runs/25948092616/job/76280307240?pr=2959

Root cause: the workflow triggers on pull_request. For PRs from forks, GitHub forces GITHUB_TOKEN to read-only regardless of the workflow's declared permissions: block. The job log confirms — it declares issues: write, pull-requests: write but the runner reports Issues: read, PullRequests: read, so the comment POST returns 403.

Net effect: the welcome comment has never worked for the audience it was meant for (first-time external contributors), and the workflow has been adding a red ❌ to every such PR. Removing it is the simplest fix. See #2964 for more discussion.

How was this patch tested?

N/A — workflow file removal only.

Did this PR include necessary documentation updates?

  • No, this PR does not affect any public API so no need to change the documentation.

The first-interaction workflow has been failing on every first-time
fork-PR with HTTP 403 "Resource not accessible by integration", because
PRs opened from forks get a read-only GITHUB_TOKEN regardless of the
workflow's declared permissions. The welcome-comment has never actually
worked for its intended audience while consistently producing a red
check on those PRs.
@jiayuasu jiayuasu requested a review from jbampton May 17, 2026 07:00
@jbampton jbampton self-assigned this May 18, 2026
@jbampton jbampton added the bug label May 18, 2026
@jbampton jbampton merged commit a313b01 into apache:master May 18, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove first-interaction CI workflow

2 participants