Skip to content

[metrics] Add PhpMetrics command (#15)#98

Merged
coisa merged 6 commits intomainfrom
codex/15-metrics-command
Apr 19, 2026
Merged

[metrics] Add PhpMetrics command (#15)#98
coisa merged 6 commits intomainfrom
codex/15-metrics-command

Conversation

@coisa
Copy link
Copy Markdown
Contributor

@coisa coisa commented Apr 19, 2026

Summary

  • simplify the metrics command to delegate directly to PhpMetrics, remove the custom metrics loader/renderer layer, and execute PhpMetrics through the active PHP binary with deprecation noise suppressed
  • align reports with optional metrics generation, keep preview-friendly output paths in the reports workflow, and include the metrics preview URL in the sticky PR comment
  • refresh README, Sphinx docs, and the local .agents/skills/package-readme guidance to cover metrics badges, links, and the current command behavior
  • update PHPUnit coverage for the new metrics and reports command flow

Validation

  • ./vendor/bin/phpunit tests/Console/Command/MetricsCommandTest.php tests/Console/Command/ReportsCommandTest.php tests/Process/ProcessBuilderTest.php
  • ./vendor/bin/phpunit tests
  • composer dev-tools metrics -- --report-html=public/metrics
  • ruby -e 'require "yaml"; YAML.load_file(".github/workflows/reports.yml"); YAML.load_file("resources/github-actions/reports.yml")'
  • git diff --check

Notes

  • the full PHPUnit run still reports an existing JoliNotif runner warning in this environment (signal 6 from the notification subscriber) after the suite completes successfully
  • preview comments now include docs, coverage, and metrics links

Closes #15

@github-actions github-actions Bot added the enhancement New feature or request label Apr 19, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 19, 2026

@coisa coisa merged commit 14b3d12 into main Apr 19, 2026
2 checks passed
@coisa coisa deleted the codex/15-metrics-command branch April 19, 2026 02:18
github-actions Bot added a commit that referenced this pull request Apr 19, 2026
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

Status: Released

Development

Successfully merging this pull request may close these issues.

Add dev-tools metrics command for code metrics analysis (PHP Metrics integration)

1 participant