Skip to content

Remove Filament Upgrade Command from Post-Autoload Scripts#1160

Merged
curtisdelicata merged 1 commit intomainfrom
sweep/Remove-Filament-Upgrade-Command-from-Post-Autoload-Scripts
May 10, 2025
Merged

Remove Filament Upgrade Command from Post-Autoload Scripts#1160
curtisdelicata merged 1 commit intomainfrom
sweep/Remove-Filament-Upgrade-Command-from-Post-Autoload-Scripts

Conversation

@sweep-ai-deprecated
Copy link
Copy Markdown
Contributor

This pull request was created by Sweep to resolve the following request by @curtisdelicata:

ERROR There are no commands defined in the "filament" namespace.

Script @php artisan filament:upgrade handling the post-autoload-dump event returned with error code 1

Continue chatting at https://app.sweep.dev/c/99bdc3ce-f9b6-437d-88d3-09ea3bc5f4cd.

Purpose

This pull request removes the Filament upgrade command from the post-autoload-dump scripts in composer.json to streamline the build process and prevent potential conflicts during package updates.

Description

The change removes the @php artisan filament:upgrade command from the post-autoload-dump scripts section in composer.json. This modification simplifies the post-autoload process by only running essential commands:

  • Package discovery
  • Composer autoload dump scripts

The removal of the Filament upgrade command allows for more controlled updates of the Filament package, letting developers explicitly run upgrades when needed rather than automatically during each autoload dump.

Summary

  • Removed @php artisan filament:upgrade from composer.json post-autoload-dump scripts
  • Maintained essential commands:
    • Illuminate\Foundation\ComposerScripts::postAutoloadDump
    • @php artisan package:discover --ansi
  • Fixed indentation in the scripts section of composer.json

@curtisdelicata curtisdelicata merged commit ec4a31c into main May 10, 2025
2 of 10 checks passed
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