Skip to content

Conversation

@poikilotherm
Copy link
Contributor

What this PR does / why we need it:
Don't run the copy label flow for PRs from forks

Which issue(s) this PR closes:

Special notes for your reviewer:
None

Suggestions on how to test this:
Open a PR from a fork, watch it not being executed

Does this PR introduce a user interface change? If mockups are available, please link/include them here:
Nope

Is there a release notes update needed for this change?:
Nope

Additional documentation:
Nope

@poikilotherm poikilotherm added Component: Code Infrastructure formerly "Feature: Code Infrastructure" Consider For Next Release A simple change (eg bug fix) that would be good to prioritize since it has been seen in the wild labels May 8, 2025
Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

Let's merge it and see if it works.

@github-project-automation github-project-automation bot moved this to Ready for QA ⏩ in IQSS Dataverse Project May 8, 2025
@ofahimIQSS ofahimIQSS self-assigned this May 8, 2025
@ofahimIQSS ofahimIQSS moved this from Ready for QA ⏩ to QA ✅ in IQSS Dataverse Project May 8, 2025
@pdurbin pdurbin added this to the 6.7 milestone May 8, 2025
@ofahimIQSS
Copy link
Contributor

Looks good, merging.

@ofahimIQSS ofahimIQSS merged commit 0d89b8f into IQSS:develop May 8, 2025
7 checks passed
@github-project-automation github-project-automation bot moved this from QA ✅ to Merged 🚀 in IQSS Dataverse Project May 8, 2025
@ofahimIQSS ofahimIQSS removed their assignment May 8, 2025
@scolapasta scolapasta moved this from Merged 🚀 to Done 🧹 in IQSS Dataverse Project May 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Component: Code Infrastructure formerly "Feature: Code Infrastructure" Consider For Next Release A simple change (eg bug fix) that would be good to prioritize since it has been seen in the wild

Projects

Status: Done 🧹

Development

Successfully merging this pull request may close these issues.

Feature Request: Stop the Copy Labels action from failing on PRs from external repositories

3 participants