Skip to content

Upgrade tooling: replace Robo with justfile, upgrade PHPUnit to v11, add CI#6

Merged
ethanhann merged 2 commits into
mainfrom
claude/upgrade-tooling-ci-VR6fG
Mar 8, 2026
Merged

Upgrade tooling: replace Robo with justfile, upgrade PHPUnit to v11, add CI#6
ethanhann merged 2 commits into
mainfrom
claude/upgrade-tooling-ci-VR6fG

Conversation

@ethanhann

Copy link
Copy Markdown
Owner
  • Replace RoboFile.php with a justfile (build, test, fmt, lint, check recipes)
  • Upgrade phpunit/phpunit from ^9 to ^11
  • Remove consolidation/robo, add phpstan/phpstan ^2.0
  • Update phpunit.xml to PHPUnit 11 format (remove deprecated attributes/listeners, coverage -> source)
  • Migrate AbstractTestCase from custom constructor to setUp() for PHPUnit 10+ compat
  • Delete TestTimeListener (TestListener interface removed in PHPUnit 10)
  • Add .github/workflows/ci.yml with fmt check, static analysis, and test jobs
  • Add CLAUDE.md with project overview and dev commands

https://claude.ai/code/session_01TAZxBvfTTM5rk51ZoJYEjm

claude added 2 commits March 8, 2026 20:43
…add CI

- Replace RoboFile.php with a justfile (build, test, fmt, lint, check recipes)
- Upgrade phpunit/phpunit from ^9 to ^11
- Remove consolidation/robo, add phpstan/phpstan ^2.0
- Update phpunit.xml to PHPUnit 11 format (remove deprecated attributes/listeners, coverage -> source)
- Migrate AbstractTestCase from custom constructor to setUp() for PHPUnit 10+ compat
- Delete TestTimeListener (TestListener interface removed in PHPUnit 10)
- Add .github/workflows/ci.yml with fmt check, static analysis, and test jobs
- Add CLAUDE.md with project overview and dev commands

https://claude.ai/code/session_01TAZxBvfTTM5rk51ZoJYEjm
Fixes 'at least one path must be specified' error when running phpstan
without explicit path arguments.

https://claude.ai/code/session_01TAZxBvfTTM5rk51ZoJYEjm
@ethanhann ethanhann merged commit 772f97b into main Mar 8, 2026
2 checks passed
@ethanhann ethanhann deleted the claude/upgrade-tooling-ci-VR6fG branch March 8, 2026 21:05
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.

2 participants