Skip to content

Conversation

@cyrusagent
Copy link

@cyrusagent cyrusagent bot commented Oct 19, 2025

Summary

  • Updated the GitHub Actions workflow to use --no-fail-fast flag with cargo nextest run
  • This ensures all tests are executed even when some fail, providing complete test suite visibility

Context

Previously, the workflow would stop running tests after the first failure. The warning message indicated:

warning: 46/125 tests were not run due to test failure (run with --no-fail-fast to run all tests, or run with --max-fail)

Changes

  • Modified .github/workflows/rust-tests.yml line 37 to add the --no-fail-fast flag

Test plan

  • The workflow will run on this PR and demonstrate running all tests regardless of failures
  • Verify in the Actions tab that all 125 tests are attempted

Fixes PER-9

🤖 Generated with Claude Code

Update the GitHub Actions workflow to use the --no-fail-fast flag with cargo nextest.
This ensures all tests run even when some fail, allowing us to see the full test suite
results instead of stopping at the first failure.

Fixes PER-9

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@weswalla weswalla merged commit 32bd92d into main Oct 19, 2025
1 check passed
@weswalla weswalla deleted the cyrus/per-9-run-all-tests-in-workflow branch October 19, 2025 15:28
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