search

LaravelRUS / SleepingOwlAdmin

🦉 Administrative interface builder for Laravel (Laravel admin)
http://sleepingowladmin.ru/
MIT License
800 stars 216 forks source link

Move IDE Helper Generator to dev dependency #832

Closed UksusoFF closed 5 years ago

UksusoFF commented 6 years ago

Он совершенно не нужен на production. В документации этого пакета его предлагается реквайрить как dev зависимость: composer require --dev barryvdh/laravel-ide-helper

zlodes commented 5 years ago

Ага, только вот есть класс SleepingOwl\Admin\Console\Generator, который наследуется от Barryvdh\LaravelIdeHelper\Generator.

Зачем так было сделано — не ясно.

ghost commented 5 years ago

@zlodes все гениальное просто, он используется для генерации идехелпер файла таким образом что бы работал автокомплит для совы.

UksusoFF commented 5 years ago

@aios ну идехелпер тоже не нужен на проде же.

zlodes commented 5 years ago

@aios, возможно, стоит генератор вынести в отдельный пакет и подключать как dev-dependency. А он уже будет зависеть от оригинального ide-helper.

UksusoFF commented 5 years ago

Можно просто вот тут не регистрировать команду с ним: https://github.com/LaravelRUS/SleepingOwlAdmin/blob/ba4ae1363fa063c529ad9c8bb4957104db5a6dbe/src/Providers/SleepingOwlServiceProvider.php#L82 Если !class_exists(\Barryvdh\LaravelIdeHelper\Generator::class)

И выше заодно убрать принудительную регистрацию IdeHelperServiceProvider.

daaner commented 5 years ago

Это уже поправлено https://github.com/LaravelRUS/SleepingOwlAdmin/blob/ba4ae1363fa063c529ad9c8bb4957104db5a6dbe/src/Providers/SleepingOwlServiceProvider.php#L67

UksusoFF commented 5 years ago

@daaner только вот он все еще в require: https://github.com/LaravelRUS/SleepingOwlAdmin/blob/development/composer.json#L46