Skip to content

T5 commands - #13

Merged
niden merged 9 commits into
masterfrom
T5-commands
Jul 2, 2026
Merged

niden merged 9 commits into
masterfrom
T5-commands

Conversation

@niden

@niden niden commented Jul 2, 2026

Copy link
Copy Markdown
Member

Hello!

In raising this pull request, I confirm the following:

  • I have read and understood the Contributing Guidelines
  • I have checked that another pull request for this purpose does not exist
  • I wrote some tests for this PR
  • I have updated the relevant CHANGELOG
  • I have created a PR for the documentation about this change

talon CLI runner: vendor/bin/talon run [suites...] [-- passthrough] fronts PHPUnit per mapped suite, talon suites lists the map. Suites come from a root talon.php (per-suite config/php/env/args, global php/env merged in) or, with zero config, from discovered phpunit*.xml files (phpunit.mysql.xml -> mysql; phpunit.xml.dist -> unit, the default). Options forward to PHPUnit starting at the first option talon does not recognize (everything after -- always forwards verbatim); the reserved name all runs every suite sequentially with a max() exit code; subprocess re-exec keeps per-suite ini flags (e.g. extension=phalcon.so) and env vars possible. Adds phalcon/cli-options-parser (^2.0) as a runtime dependency.

The repo dogfoods its own runner: composer test/test-coverage/test-coverage-html and the CI database step now route through bin/talon (php bin/talon run mysql pgsql replaces the two explicit PHPUnit invocations). Infection still drives PHPUnit directly by design; the raw vendor/bin/phpunit -c ... invocations keep working as a fallback.

Thanks

niden added 8 commits July 2, 2026 14:58
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
@niden
niden requested a review from Jeckerson July 2, 2026 20:10
@niden niden self-assigned this Jul 2, 2026
@niden niden added the enhancement New feature or request label Jul 2, 2026
@niden niden linked an issue Jul 2, 2026 that may be closed by this pull request
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Assisted-by: Claude Code
@github-actions

This comment has been minimized.

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

Code Metrics Report

master (ef75567) T5-commands (685ccff) +/-
Coverage 100.0% 100.0% 0.0%
Test Execution Time 1s 1s 0s
Details
  |                     | master (ef75567) | T5-commands (685ccff) | +/-  |
  |---------------------|------------------|-----------------------|------|
  | Coverage            |           100.0% |                100.0% | 0.0% |
  |   Files             |               38 |                    46 |   +8 |
  |   Lines             |              588 |                   774 | +186 |
+ |   Covered           |              588 |                   774 | +186 |
  | Test Execution Time |               1s |                    1s |   0s |

Code coverage of files in pull request scope (0.0% → 100.0%)

Files Coverage +/- Status
src/Cli/Command/RunCommand.php 100.0% +100.0% added
src/Cli/Command/SuitesCommand.php 100.0% +100.0% added
src/Cli/Input.php 100.0% +100.0% added
src/Cli/Kernel.php 100.0% +100.0% added
src/Cli/ProcessRunner.php 100.0% +100.0% added
src/Cli/Suite.php 100.0% +100.0% added
src/Cli/SuiteMap.php 100.0% +100.0% added
src/Exceptions/UnknownSuite.php 100.0% +100.0% added

Reported by octocov

@niden
niden merged commit 8de5926 into master Jul 2, 2026
22 checks passed
@niden
niden deleted the T5-commands branch July 2, 2026 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[NFR] Commands

1 participant