Skip to content

Helpers#1

Open
elukianov wants to merge 15 commits intoBukharovsi:masterfrom
stringconcat:helpers
Open

Helpers#1
elukianov wants to merge 15 commits intoBukharovsi:masterfrom
stringconcat:helpers

Conversation

@elukianov
Copy link

No description provided.

Copy link
Author

@elukianov elukianov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Отличная работа! Хотелось бы добавить несколько моментов, которые требуют доработки:

  • Было бы неплохо добавить стат анализатор в процесс сборки на начальном этапе. Как мы уже говорили в наших лекциях, на более позднем этапе никто не будет обращать на него внимания, потому что он будет подсвечивать миллион ошибок. Разобраться в таком будет непросто. В лучшем случае кто-то будет вяло пытаться починить найденное, в худшем - просто выкинут этот анализатор из проекта.

  • Из README не совсем понятно, какой именно артефакт мы получим в результате работы этой “кнопки”. Что это будет? JAR, WAR, Docker-образ или же ещё что-то? Хотелось бы увидеть более конкретное описание

  • После запуска тестов не генерируется jacoco-отчет, хотя он указан в плагинах. Возможно имеют место быть некорректные настройки. Нужно разобраться в причинах

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants