Как мы делали Linux-пингвинов к Новому году

[@andrewnester]

есть пару замечаний, которые сходу бросились в глаза
1) папка vendor и composer.lock в репозитории
2) местами разный стиль кодирования, для open-source проектов лучше выбирайте PSR
3) нет тестов
4) местами макаронный код github.com/ZENLIX/LaraShop/blob/master/laravel/app/Http/Controllers/PurchaseController.php

ну и уже как личное предпочтение, в Laravel 5 хороший DI механизм, лично я предпочитаю его фасадам, как по мне статические фасады в Laravel было не самой классной идеей

[@AotD]

vendor таки да, нафиг, а что не так с composer.lock? Версии зафиксированы, никаких неожиданностей.

[@andrewnester]

ну если рассматривать этот код как код исключительно проекта/приложения, то всё ок, согласен

честно говоря, лучше было бы оформить данный код как Laravel package, а не цельное приложение и вот там лично я бы composer.lock убрал, как советует доки по composer

[@andrewnester]

и смотрели ли Вы на готовые решения, например, github.com/amsgames/laravel-shop

[@Alex10]

А фильтр по атрибутам товаров есть?

[@Pinsky]

У Вас мобильная вверстка поехала.

[@thunderspb]

Только у меня не сходится название и содержание? Или я что-то такое не понимаю? Если второе, то объясните пожалуйста :)