Skip to content

T10 runtime - #23

Merged
niden merged 4 commits into
masterfrom
T10-runtime
Sep 26, 2026
Merged

niden merged 4 commits into
masterfrom
T10-runtime

Conversation

@niden

@niden niden commented Sep 25, 2026

Copy link
Copy Markdown
Member

Hello!

In raising this pull request, I confirm the following:

  • I have checked that another pull request for this purpose does not exist
  • I wrote some tests for this PR
  • composer test, composer cs and composer analyze all pass
  • I have created a PR for the documentation about this change, if user facing

  • Added serve (alias server), running PHP's built-in web server in the project root with the router script that new writes: php -S 127.0.0.1:8080 -t public .htrouter.php, with the same router and document root as the generated container. The port is --port, else APP_PORT from the environment or the project .env, else 8080, the order that docker compose uses. It finds the root from a subdirectory through the nearest crest.php, and it does not need the crest in vendor/. It stops before PHP starts when .htrouter.php or vendor/autoload.php is missing. new and the generated README now print crest serve for the host way.

Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
Assisted-by: Claude Code
@niden
niden requested a review from Jeckerson September 25, 2026 23:57
@niden niden self-assigned this Sep 25, 2026
@github-actions

This comment has been minimized.

@github-actions

Copy link
Copy Markdown

Code Metrics Report

master (6e58901) #23 (80d5e96) +/-
Coverage 98.9% 99.0% +0.0%
Test Execution Time 1s 1s 0s
Details
  |                     | master (6e58901) | #23 (80d5e96) |  +/-  |
  |---------------------|------------------|---------------|-------|
+ | Coverage            |            98.9% |         99.0% | +0.0% |
  |   Files             |               48 |            49 |    +1 |
  |   Lines             |             1070 |          1116 |   +46 |
+ |   Covered           |             1059 |          1105 |   +46 |
  | Test Execution Time |               1s |            1s |    0s |

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

Files Coverage +/- Patch Coverage Status
src/Command/NewCommand.php 100.0% 0.0% 100.0% modified
src/Command/ServeCommand.php 100.0% +100.0% 100.0% added
src/Commands.php 100.0% 0.0% 100.0% modified

Reported by octocov

@niden
niden merged commit 17d3e85 into master Sep 26, 2026
24 checks passed
@niden
niden deleted the T10-runtime branch September 26, 2026 00:06
@niden niden mentioned this pull request Sep 26, 2026
5 tasks
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