Skip to content

ci: add PR test workflow#7

Closed
nbucic wants to merge 5 commits intoclaude/android-interval-timer-app-JYhZcfrom
bump-version
Closed

ci: add PR test workflow#7
nbucic wants to merge 5 commits intoclaude/android-interval-timer-app-JYhZcfrom
bump-version

Conversation

@nbucic
Copy link
Copy Markdown
Owner

@nbucic nbucic commented Apr 13, 2026

Summary

  • Adds .github/workflows/run-tests.yml that triggers on every pull request
  • Sets up PHP 8.5 with zip, sqlite3, and pdo_sqlite extensions
  • Caches Composer dependencies keyed on composer.lock for faster reruns
  • Runs the full Pest suite via composer test (78 unit tests, SQLite in-memory)

Test plan

  • Open a PR and verify the Run Tests check appears and passes
  • Push a new commit to an open PR and verify the check reruns

nbucic and others added 5 commits April 10, 2026 16:20
* fix: resolve timer not starting and display bugs in TimerScreen
* fix: use PHP 8.5 clone() syntax in resumeAs()
* fix: Minor Timer Runner and Screen fixes
* fix: timer iteration
* feat: history log, ring UI, PREPARE state, and long-press pause
Runs the Pest suite on every pull request (opened, synchronize, reopened)
using PHP 8.5, SQLite in-memory, and a Composer cache for fast reruns.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Pest could not resolve Tests\TestCase because the namespace was missing
from composer.json autoload-dev mappings.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@nbucic nbucic closed this Apr 13, 2026
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