Skip to content

[chore] Add repository check to tidy-dependencies workflow - #1712

Merged
drewrelmas merged 2 commits into
open-telemetry:mainfrom
dmitryax:block-forks-in-tidy-workflow
Jan 5, 2026
Merged

[chore] Add repository check to tidy-dependencies workflow#1712
drewrelmas merged 2 commits into
open-telemetry:mainfrom
dmitryax:block-forks-in-tidy-workflow

Conversation

@dmitryax

@dmitryax dmitryax commented Jan 3, 2026

Copy link
Copy Markdown
Member

Add a repository origin check to the tidy.yml workflow (with write permissions) to ensure it can't be triggered by PRs from forks.

Same as open-telemetry/opentelemetry-collector#14344

Adds a repository origin check to the tidy-dependencies.yml workflow condition to ensure isn't triggered by PRs from forks.
@dmitryax
dmitryax requested a review from a team as a code owner January 3, 2026 20:04
@github-actions github-actions Bot added the ci-repo Repository maintenance, build, GH workflows, repo cleanup, or other chores label Jan 3, 2026
@codecov

codecov Bot commented Jan 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.07%. Comparing base (32a6fbb) to head (29318ab).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1712      +/-   ##
==========================================
- Coverage   84.08%   84.07%   -0.02%     
==========================================
  Files         469      469              
  Lines      137651   137651              
==========================================
- Hits       115746   115729      -17     
- Misses      21371    21388      +17     
  Partials      534      534              
Components Coverage Δ
otap-dataflow 85.33% <ø> (-0.02%) ⬇️
query_abstraction 80.61% <ø> (ø)
query_engine 90.39% <ø> (ø)
syslog_cef_receivers ∅ <ø> (∅)
otel-arrow-go 53.50% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@albertlockett albertlockett left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks @dmitryax

@cijothomas cijothomas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@drewrelmas
drewrelmas enabled auto-merge January 5, 2026 17:59
@drewrelmas
drewrelmas added this pull request to the merge queue Jan 5, 2026
Merged via the queue into open-telemetry:main with commit 3309c75 Jan 5, 2026
43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-repo Repository maintenance, build, GH workflows, repo cleanup, or other chores

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants