Skip to content

Add GitHub Actions workflow for Laravel tests #1

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
shash-hq wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add GitHub Actions workflow for Laravel tests #1

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 35 additions & 0 deletions .github/workflows/laravel.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
name: Laravel

on:
push:
branches: [ "main" ]
pull_request:
branches: [ "main" ]

jobs:
laravel-tests:

runs-on: ubuntu-latest

steps:
- uses: shivammathur/setup-php@15c43e89cdef867065b0213be354c2841860869e
with:
php-version: '8.0'
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot Mar 16, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1: This workflow installs PHP 8.0, but the project requires PHP ^8.2, so composer install will fail and the test job won't run.

Prompt for AI agents 
Check if this issue is valid — if so, understand the root cause and fix it. At .github/workflows/laravel.yml, line 17:

<comment>This workflow installs PHP 8.0, but the project requires PHP ^8.2, so `composer install` will fail and the test job won't run.</comment>

<file context>
@@ -0,0 +1,35 @@
+    steps:
+    - uses: shivammathur/setup-php@15c43e89cdef867065b0213be354c2841860869e
+      with:
+        php-version: '8.0'
+    - uses: actions/checkout@v4
+    - name: Copy .env
</file context>
Fix with Cubic

- uses: actions/checkout@v4
- name: Copy .env
run: php -r "file_exists('.env') || copy('.env.example', '.env');"
- name: Install Dependencies
run: composer install -q --no-ansi --no-interaction --no-scripts --no-progress --prefer-dist
Comment on lines +17 to +22
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot Mar 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "Workflow PHP version:"
rg -nP "php-version:\s*['\"][0-9.]+" .github/workflows/laravel.yml

echo "Composer PHP constraint:"
rg -nP '"php"\s*:\s*"[^"]+"' composer.json

Repository: shash-hq/lore

Length of output: 158

PHP version incompatible with project requirements (CI blocker)

Line 17 sets PHP 8.0, but composer.json requires ^8.2. The composer install step will fail because PHP 8.0 does not satisfy the constraint.

Proposed fix 
-        php-version: '8.0'
+        php-version: '8.2'
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
php-version: '8.0'
- uses: actions/checkout@v4
- name: Copy .env
run: php -r "file_exists('.env') || copy('.env.example', '.env');"
- name: Install Dependencies
run: composer install -q --no-ansi --no-interaction --no-scripts --no-progress --prefer-dist
php-version: '8.2'
- uses: actions/checkout@v4
- name: Copy .env
run: php -r "file_exists('.env') || copy('.env.example', '.env');"
- name: Install Dependencies
run: composer install -q --no-ansi --no-interaction --no-scripts --no-progress --prefer-dist
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In @.github/workflows/laravel.yml around lines 17 - 22, The CI workflow sets
php-version: '8.0' which doesn't meet composer.json's ^8.2 requirement; update
the workflow's PHP version to a compatible version (e.g., change php-version to
'8.2' or a matrix including '8.2'/'8.3') so the Install Dependencies step (the
run executing composer install) runs under a PHP that satisfies the constraint;
keep the existing actions/checkout, Copy .env, and Install Dependencies steps
unchanged except for this php-version update.

- name: Generate key
run: php artisan key:generate
- name: Directory Permissions
run: chmod -R 777 storage bootstrap/cache
- name: Create Database
run: |
mkdir -p database
touch database/database.sqlite
- name: Execute tests (Unit and Feature tests) via PHPUnit/Pest
env:
DB_CONNECTION: sqlite
DB_DATABASE: database/database.sqlite
run: php artisan test
Comment on lines +15 to +35
Loading