Conversation
| @@ -0,0 +1,18 @@ | |||
| # Licensed to the Apache Software Foundation (ASF) under one | |||
There was a problem hiding this comment.
в репозитории должно быть только то, что находится внутри твоего проекта, Т.е. рутовая помка и все что находится на ее уровне. У тебя почему-то tasks/maven/hw - так не должно быть
| <properties> | ||
| <maven.compiler.source>15</maven.compiler.source> | ||
| <maven.compiler.target>15</maven.compiler.target> | ||
| <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding> |
There was a problem hiding this comment.
properties тоже наследуются, как и dependency, plugins, etc. Поэтому достаточно определить их лишь раз в parent pom
| <dependency> | ||
| <groupId>org.json</groupId> | ||
| <artifactId>json</artifactId> | ||
| <version>20230227</version> |
There was a problem hiding this comment.
не должно быть лишних зависимостей в проекте
| <dependencies> | ||
| <dependency> | ||
| <groupId>junit</groupId> | ||
| <artifactId>junit</artifactId> |
There was a problem hiding this comment.
по заданию нужно было junit 5 использовать
| <app.env>dev</app.env> | ||
| <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding> | ||
| <maven.compiler.source>1.8</maven.compiler.source> | ||
| <maven.compiler.target>1.8</maven.compiler.target> |
| <groupId>com.dmdev</groupId> | ||
| <artifactId>common</artifactId> | ||
| <version>1.0.0-SNAPSHOT</version> | ||
| <scope>compile</scope> |
There was a problem hiding this comment.
compile по умолчанию - не нужно указывать явно
| <plugins> | ||
| <plugin> | ||
| <groupId>org.apache.maven.plugins</groupId> | ||
| <artifactId>maven-assembly-plugin</artifactId> |
There was a problem hiding this comment.
по заданию не нужно было. Поэтому можно было опустить)
| <mainClass>com.dmdev.service.AppRunner</mainClass> | ||
| </manifest> | ||
| </archive> | ||
| </configuration> |
There was a problem hiding this comment.
не определил jacoco plugin - работать не будет (в parent pom он в pluginManagement - а значит не будет применяться)
| import com.dmdev.common.repository.UserRepository; | ||
| import com.dmdev.common.models.User; | ||
|
|
||
|
|
There was a problem hiding this comment.
что-то не то с отступами между импортами/классами
их как-то много, как будто вручную правил
| @@ -0,0 +1,2 @@ | |||
| public class AppIT { | |||
| } | |||
There was a problem hiding this comment.
надо было хотя бы по одному тесту добавить, чтобы проверить - что работает
2 participants
No description provided.