Skip to content

Conversation

@seisman
Copy link
Member

@seisman seisman commented Oct 31, 2025

@Chuan1937 just opened his first PR (#4180). The Greeting workflow added in PR #4149 should have posted a welcome message but didn’t.

The workflow run failed with the error: “Resource not accessible by integration” (https://github.com/GenericMappingTools/pygmt/actions/runs/18977980962/job/54206345492).

This is related to the upstream issue actions/first-interaction#10. The PR applies the solution suggested at actions/first-interaction#10 (comment).

I haven't tested whether it works and don't plan to spend time on it. We'll find out when the next new contributor arrives.

@seisman seisman added this to the 0.18.0 milestone Oct 31, 2025
@seisman seisman added maintenance Boring but important stuff for the core devs skip-changelog Skip adding Pull Request to changelog needs review This PR has higher priority and needs review. labels Oct 31, 2025
contents: read

jobs:
greeting:
Copy link
Member

Choose a reason for hiding this comment

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

Can we add an if-condition to only run this on forks? First time contributors usually won't run on 'GenericMappingTools/pygmt' unless they're a bot or something.

Copy link
Member Author

Choose a reason for hiding this comment

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

In aae7d87, I've added the if-condition

if: github.event_name == 'issues' || (github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name != github.repository)

@seisman seisman merged commit 4c7e356 into main Nov 4, 2025
8 checks passed
@seisman seisman deleted the ci/fix-welcome branch November 4, 2025 03:17
@seisman seisman removed the needs review This PR has higher priority and needs review. label Nov 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Boring but important stuff for the core devs skip-changelog Skip adding Pull Request to changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants