Skip to content

Conversation

@martincostello
Copy link
Member

Changes

Skip labelling on forks.

I accidentally opened a PR in my fork, then noticed that the labelling job failed because my fork doesn't contain this repo's labels.

Merge requirement checklist

  • CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
  • Unit tests added/updated
  • Appropriate CHANGELOG.md files updated for non-trivial changes
  • Changes in public API reviewed (if applicable)

Skip labelling PRs on forks.
@martincostello martincostello requested a review from a team as a code owner December 12, 2025 11:03
Copilot AI review requested due to automatic review settings December 12, 2025 11:03
@github-actions github-actions bot added the infra Infra work - CI/CD, code coverage, linters label Dec 12, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds fork detection to the label workflow to prevent failures when pull requests or issues are opened in forked repositories that don't have the same label configuration as the upstream repository.

  • Added github.event.repository.fork == false condition to both labeling jobs
  • Prevents workflow failures in forks due to missing labels

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link

codecov bot commented Dec 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.69%. Comparing base (c6a203f) to head (b83626e).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6765      +/-   ##
==========================================
- Coverage   86.73%   86.69%   -0.05%     
==========================================
  Files         262      262              
  Lines       12319    12319              
==========================================
- Hits        10685    10680       -5     
- Misses       1634     1639       +5     
Flag Coverage Δ
unittests-Project-Experimental 86.38% <ø> (-0.30%) ⬇️
unittests-Project-Stable 86.66% <ø> (ø)
unittests-Solution 86.58% <ø> (-0.20%) ⬇️
unittests-UnstableCoreLibraries-Experimental 86.07% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 2 files with indirect coverage changes

@Kielek Kielek added this pull request to the merge queue Dec 12, 2025
Merged via the queue into open-telemetry:main with commit 3202b94 Dec 12, 2025
91 checks passed
@martincostello martincostello deleted the skip-labels-on-forks branch December 12, 2025 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra Infra work - CI/CD, code coverage, linters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants