Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set up Tests GitHub Action workflow with tmate debugging #364

Merged
merged 1 commit into from
May 3, 2024

Conversation

candleindark
Copy link
Collaborator

Set up tmate debugging for the Tests GitHub Action workflow to be triggered manually

Copy link

codecov bot commented May 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.77%. Comparing base (5b1e274) to head (4dfe9fa).
Report is 2 commits behind head on master.

❗ Current head 4dfe9fa differs from pull request most recent head 9a08e32. Consider uploading reports for the commit 9a08e32 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #364   +/-   ##
=======================================
  Coverage   98.77%   98.77%           
=======================================
  Files          55       55           
  Lines        2605     2605           
=======================================
  Hits         2573     2573           
  Misses         32       32           

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

@candleindark candleindark requested a review from yarikoptic May 3, 2024 01:31
@candleindark
Copy link
Collaborator Author

candleindark commented May 3, 2024

For the Tests workflow to be manually triggered to run on this repo, these changes have to be in a branch of the repo. So until these changes are merged, the Tests workflow can't be triggered to run on this repo. I was, however, able to trigger the Tests workflow to run on my fork using gh workflow run Tests -r setup-tmate -f debug_enabled=true and access the host system of the workflow action. Thus, this PR should be good to merge.

@candleindark candleindark marked this pull request as draft May 3, 2024 04:29
@candleindark
Copy link
Collaborator Author

I will put in a fail trigger as well tomorrow.

@candleindark candleindark force-pushed the setup-tmate branch 2 times, most recently from 4dfe9fa to 9a08e32 Compare May 3, 2024 18:36
@candleindark
Copy link
Collaborator Author

I will put in a fail trigger as well tomorrow.

Never mind, triggering a tmate session on each failure can be too costly in terms of GitHub Actions usage. Let's stay with just the manual trigger.

@candleindark candleindark marked this pull request as ready for review May 3, 2024 18:45
@candleindark candleindark merged commit 7ede3ae into datalad:master May 3, 2024
10 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.

1 participant