Skip to content

fossa: update the check to skip running in forks - #486

Merged
trask merged 1 commit into
open-telemetry:mainfrom
codeboten:codeboten/fossa-guard-forks
Oct 21, 2026
Merged

trask merged 1 commit into
open-telemetry:mainfrom
codeboten:codeboten/fossa-guard-forks

Conversation

@codeboten

Copy link
Copy Markdown
Member

This is a check that was done in many repos with "github.repository == 'open-telemetry/opentelemetry.io'", using the fork value is more a more generic way to achieve the same

This is a check that was done in many repos with "github.repository == 'open-telemetry/opentelemetry.io'", using the fork value is more a more generic way to achieve the same

Signed-off-by: Alex Boten <223565+codeboten@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 20, 2026 17:45
@codeboten
codeboten requested a review from a team as a code owner October 20, 2026 17:45
@codeboten
codeboten requested a review from trask October 20, 2026 17:45
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Pull request dashboard status

Merged · refreshed 2026-10-21 17:41 UTC

Status above doesn't look right?
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The condition excludes fork repositories while preserving the existing ownership restriction.

No open issues 🎉

What changed in this PR

Updates the FOSSA workflow to skip fork repositories while retaining the existing organization restriction.

Changes:

  • Requires the repository to belong to open-telemetry and not be a fork before scanning.
File Description
.github/​workflows/​fossa.yml Adds a fork check to the FOSSA job condition.

Review effort: Balanced

@trask
trask merged commit f5862c3 into open-telemetry:main Oct 21, 2026
6 checks passed
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.

5 participants