Skip to content

Conversation

@tjeerddie
Copy link
Contributor

  • Change get_auth_callbacks to work with optional workflow param
  • Convert values to boolean for retryAllowed and resumeAllowed for graphql process user_permissions

@codspeed-hq
Copy link

codspeed-hq bot commented Nov 4, 2025

CodSpeed Performance Report

Merging #1162 will not alter performance

Comparing fix-errors-for-visible-soft-deleted-workflows (415a291) with main (08451d1)

Summary

✅ 13 untouched

@tjeerddie tjeerddie force-pushed the fix-errors-for-visible-soft-deleted-workflows branch from c318daa to b7962fd Compare November 4, 2025 13:30
@tjeerddie tjeerddie force-pushed the fix-errors-for-visible-soft-deleted-workflows branch from efc9f6b to 49c9874 Compare November 4, 2025 14:44
@sentry
Copy link

sentry bot commented Nov 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.23%. Comparing base (08451d1) to head (415a291).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1162      +/-   ##
==========================================
+ Coverage   80.19%   80.23%   +0.04%     
==========================================
  Files         266      266              
  Lines       12848    12850       +2     
  Branches     1256     1257       +1     
==========================================
+ Hits        10303    10310       +7     
+ Misses       2265     2260       -5     
  Partials      280      280              

☔ 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.

@pboers1988 pboers1988 merged commit 9f6a4e5 into main Nov 5, 2025
16 checks passed
@pboers1988 pboers1988 deleted the fix-errors-for-visible-soft-deleted-workflows branch November 5, 2025 19:58
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.

4 participants