Skip to content

Conversation

@williscool
Copy link
Owner

@williscool williscool commented Jan 22, 2026

Make tmate opt-in for the main CI workflow (actions.yml) to reduce noise during normal runs, while keeping it available for manual debugging.


Open in Cursor Open in Web


Note

Makes tmate debugging opt-in for the main CI workflow.

  • Adds workflow_dispatch.inputs.enable_tmate (boolean, default false)
  • Updates tmate steps in build, verify-connected-android-test-working-single-integration-test, and integration-test to run only when failure() && inputs.enable_tmate == true

Written by Cursor Bugbot for commit bf64a04. This will update automatically on new commits. Configure here.

- Add enable_tmate boolean input (default: false) to workflow_dispatch
- Update all 3 tmate steps to only run when input is enabled AND failure occurs
- Normal CI runs (push/PR) no longer trigger tmate on failure
- Manual runs can enable tmate by checking the input when debugging is needed
- Leave release.yml and version-bump.yml unchanged (rare failures, worth keeping)

Co-authored-by: wharris <wharris@upscalews.com>
@cursor
Copy link

cursor bot commented Jan 22, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@williscool williscool marked this pull request as ready for review January 22, 2026 01:14
@williscool williscool changed the title CI tmate configuration ci: tmate configuration Jan 22, 2026
@github-actions
Copy link

Build artifacts for PR #232 (commit 6f619cb) are available:

You can download these artifacts from the "Artifacts" section of the workflow run.

@williscool williscool merged commit 88a6aca into master Jan 22, 2026
15 checks passed
@github-actions
Copy link

Code Coverage Report

Overall Project 34.92% 🍏

There is no coverage information present for the Files changed

@github-actions
Copy link

📊 Code Coverage Summary

Coverage Type Coverage
Overall 34.92
Changed Files 100

View detailed coverage report

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.

3 participants