Skip to content

Add FilaCheck to larasets PHP workflow stubs#49

Merged
tomgrv merged 3 commits into
developfrom
copilot/add-filacheck-to-larasets
May 18, 2026
Merged

Add FilaCheck to larasets PHP workflow stubs#49
tomgrv merged 3 commits into
developfrom
copilot/add-filacheck-to-larasets

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 18, 2026

Larasets’ PHP workflow stubs now include FilaCheck alongside the existing PHP checks, so Filament-specific issues are covered in both PR validation and auto-fix flows.

  • PR validation

    • Adds tomgrv/actions/run-filacheck@v1 to validate-pr-php.yml
    • Runs against app/Filament and only when the project includes Filament
  • Auto-fix workflow

    • Adds a FilaCheck fix step to fix-php.yml
    • Expands PR generation conditions to include FilaCheck output
    • Keeps generated fix PRs aligned with all fix-capable PHP checks
    • Only runs the FilaCheck fix step when app/Filament exists

Example:

- name: Run FilaCheck
  uses: tomgrv/actions/run-filacheck@v1
  with:
    github-token: ${{ secrets.GITHUB_TOKEN }}
    path: app/Filament

Copilot AI linked an issue May 18, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Add filacheck in larasets as additional action Add FilaCheck to larasets PHP workflow stubs May 18, 2026
Copilot finished work on behalf of tomgrv May 18, 2026 16:46
Copilot AI requested a review from tomgrv May 18, 2026 16:46
Copy link
Copy Markdown
Owner

@tomgrv tomgrv left a comment

Choose a reason for hiding this comment

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

Should run only if filament-enabled project.

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 18, 2026

Should run only if filament-enabled project.

Done in f20a876: FilaCheck now only runs when app/Filament exists, so it stays scoped to Filament-enabled projects.

Copilot AI requested a review from tomgrv May 18, 2026 16:53
Copilot finished work on behalf of tomgrv May 18, 2026 16:53
@tomgrv tomgrv marked this pull request as ready for review May 18, 2026 17:37
@tomgrv tomgrv merged commit 178df86 into develop May 18, 2026
@tomgrv tomgrv deleted the copilot/add-filacheck-to-larasets branch May 18, 2026 17:38
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.

add filacheck in larasets

2 participants