Skip to content

CompatHelper: inherit secrets in reusable caller#36

Merged
mtfishman merged 1 commit intomainfrom
mf/compathelper-secrets-inherit
May 1, 2026
Merged

CompatHelper: inherit secrets in reusable caller#36
mtfishman merged 1 commit intomainfrom
mf/compathelper-secrets-inherit

Conversation

@mtfishman
Copy link
Copy Markdown
Member

Summary

  • add secrets: "inherit" to .github/workflows/CompatHelper.yml
  • keep workflow behavior unchanged otherwise

Why

This lets the reusable CompatHelper workflow receive COMPATHELPER_PAT when available, so token-owner detection uses the PAT owner path (ITensorBot) instead of falling back to the GitHub App token path.

Scope

  • minimal one-line workflow caller patch
  • prep step so repos are in a common state ahead of the broader security-workflow rollout

@mtfishman mtfishman merged commit edc5ae1 into main May 1, 2026
11 checks passed
@mtfishman mtfishman deleted the mf/compathelper-secrets-inherit branch May 1, 2026 16:33
@codecov
Copy link
Copy Markdown

codecov Bot commented May 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.19%. Comparing base (323c1aa) to head (e7b6030).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #36   +/-   ##
=======================================
  Coverage   75.19%   75.19%           
=======================================
  Files           7        7           
  Lines         129      129           
=======================================
  Hits           97       97           
  Misses         32       32           
Flag Coverage Δ
docs 0.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant