hackgvl / hackgreenville-com

HackGreenville's Website
https://hackgreenville.com
MIT License
16 stars 15 forks source link

Remove Telescope auto-loading from config/app.php since it's now loading dynamically #243

Closed allella closed 1 month ago

allella commented 1 month ago

242 now loads app/Providers/AppServiceProvider.php if Telescope is enabled in the .env environment variables.

This autoloading was added before we decided to allow Telescope to be run based on configuration.

Since we use composer install --no-dev on stage and production, Telescope is not installed and this would cause a composer error as previously documented.

Fixeds the Script @php artisan package:discover handling the post-autoload-dump event returned with error code 1 Composer issue.

allella commented 1 month ago

Merged this to fix a breaking deploy.

If there's a better solution, then please holler. Though, it did fix the error and seems to be a logical addition to #242