именование файлов, блоков, миксинов в разных кейсах где-то camelCase, где-то lower-case, где-то вообще не понятный. Нужно все привести к одному формату 'lower-case'.
А вот именование переменных должно быть наоборот только в camelCase Потому что по сути это js переменные и им соответствуют правила для js
Зависимости, которые попадают в конечный исходный код должны быть прописаны в dependencies, а те которые нужны для разработки в devDependencies. Сейчас у тебя в зависимостях есть даже vue-loader)
Мне не нравится все равно что в папке pug лежит папка common, а в ней лежат глобальные стили, и нет пага самого. Предлагаю в целом избавиться от папки pug. Тк в целом внутри структура неплохая.
Блоки должны быть универсальными и не влиять на другие блоки, у тебя сейчас в одном блоке задаются стили для другого, такого быть не должно
Миксы у нас запрещены в стандартах
Классы не везде семантичные, называть класс по типу first-ul, secondary-ul - неправильно
от блока Buttons не вижу никакого смысла, он нигде переиспользоваться не может. Суть блока в том, что он может быть переиспользован, пройдись по проекту, посмотри какие блоки можно, а какие нет смысла где-то использовать еще кроме текущего места.
Привет, я твой ревьюер! чтобы связаться со мной можешь писать мне в телеграмм.
Не очень ясна структура папок.
в pug запихано абсолютно все по проекту, даже подключаемые библиотеки, хотя им явное место в папке libs. Сразу по ним замечание. Зачем было выкачивать абсолютно всю ветку этих либ? если тебе по сути нужен только исполняемый скрипт.
Также непонятно зачем создавать папку includes а в ней modules. Лучше просто наименовать её components, или modules.
layout выглядит как свалка всего чему не нашел место. И переменным и миксинам и тд. В layout должен лежать pug и scss (и если нужно то js) для этого шаблона. Если их несколько - нужно раздробить по папкам. А все глобальное вынести в другое место.
Также непонятно зачем в папке pug utils лежит. Эти utils состоят из js файла, к pug они не имеют никакого отношения.
На всех кликабельных элементах должен быть cursor: pointer
в пагинации цифры какие-то слишком маленькие, мне кажется на макете они больше. Так же видно, что стрелка не соотносится с макетом. И эффект активного элемента пагинации не тот
Календарь тоже не тот
Проверь еще раз пожалуйста на соответствие макету, имеются явные ошибки. Чтобы не тратить своё и моё время) Так процесс ревью будет гораздо быстрее
Блоки должны состоять из миксинов и не вызваться в самом блоке, а на странице где этот блок нужен. Сейчас у тебя есть блоки которые через include выполняются, какие то через миксины реализованы, но вызываются в самом блоке.
Советую поставить тебе code spell checker для проверки на орфографию. По дефолту он идет с англ словарем, можно добавить еще и русский язык. И поправь его ошибки пожалуйста. На будущее, можешь сделать это сразу и на слайдере)