Добавить PMD-плагин

KruASe76 / news-recommendation-service

Part of education at MIPT, HSSE under the lead of MTS

0 stars 0 forks source link

Добавить PMD-плагин #2

Closed AlexanderBobryakov closed 3 weeks ago

AlexanderBobryakov commented 1 month ago

TO-BE:

Плагин https://docs.gradle.org/current/userguide/pmd_plugin.html
Проверить, что он запускается при build проекта и проверяет правильность написания кода (для эксперимента можно допустить какую-нибудь ошибку)
Оповестить команду об этом плагине

AlexanderBobryakov commented 1 month ago

В коде остался sout, на который pmd-плагин должен ругаться. На данный момент в проекте pmd работает в минимальном составе, необходимо добавить существующие ruleSets (как минимум category/java/bestpractices.xml), см https://docs.gradle.org/current/dsl/org.gradle.api.plugins.quality.PmdExtension.html

Также все мр нужно скидывать на ревью, а не просто закрывать. Мне нужно оценивать работу каждого члена команды на протяжении всей работы

KruASe76 commented 1 month ago

необходимо добавить существующие ruleSets

окей, сейчас сделаю

Также все мр нужно скидывать на ревью, а не просто закрывать.

то есть всегда ставить тебя единственным ревьювером?

AlexanderBobryakov commented 1 month ago

то есть всегда ставить тебя единственным ревьювером?

В идеале, чтобы все члены команды смотрели пулреквесты и ставили апрувы, но тут смотрите по вашей загрузке. Меня точно нужно в ревьюверы