1mavic / to-do-app

simple to-do-app
0 stars 0 forks source link

Ревью ДЗ4 #7

Open medvalna opened 1 year ago

medvalna commented 1 year ago

Привет! Оценка работы по критериям:

1)Readme соответствует критериям +1 балл 2)flutter_lints: проверка dart analyze выдала 2200 игноров правил +0 баллов 3)форматирование кода присутствует + 2 балла 4)код разделен на слои + 2 балла 5)инкапсуляции навигации есть +1 балл 6)обрезка заметок по макету есть +1 балл 7)ночной тема есть +1 балл 8) анимация для добавления и редактирования заметок есть +1 балл 9)поддержка landscape-ориентации есть +2 балла 10)поддержка больших экранов есть +2 балла 11)работа с remote configs есть +2 балла 12)работа с firebase crachlitics есть +1 балл 13)работа с флейворами есть +2 балла 14)CI на github есть + 4 балла 15)работа с firebase app distribution есть + 2 балла 16)работа с firebase analytics есть + 1 балл 17)пакет freezed для дата-моделей используется + 1 баллов. Итог: 22 балла

1mavic commented 1 year ago

@MedVAL-na добрый день! Спасибо за ревью, подскажите пожалуйста, какой командой именно проводили анализ на игнор правил? И за 4 пункт макс 1 балла только можно вроде

medvalna commented 1 year ago

Команда dart analyze. За 4 пункт 2 балла максимум, в третьем действительно ошибка, там максимум 1. Еще я обнаружила ошибку в подсчёте баллов, в итоге у вас 23.

1mavic commented 1 year ago

@MedVAL-na

Снимок экрана 2023-07-20 в 21 37 58
medvalna commented 1 year ago

Хм, странно, но окей, подняла балл