-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: lifecycle miranum-deployment #444
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #444 +/- ##
=========================================
Coverage 58.21% 58.21%
Complexity 60 60
=========================================
Files 38 38
Lines 560 560
Branches 50 50
=========================================
Hits 326 326
Misses 185 185
Partials 49 49
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! 👍
Please have a look at my comments before merging
<groupId>org.hamcrest</groupId> | ||
<artifactId>hamcrest-all</artifactId> | ||
<version>1.3</version> | ||
<groupId>org.springframework.boot</groupId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
magst du die nicht in die parent-pom legen? Die verwendest du oben auch.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nicht jedes Modul hat Spring Boot als Dependency. Deswegen steht das nicht im parent
</resources> | ||
<plugins> | ||
<plugin> | ||
<groupId>org.springframework.boot</groupId> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hast du geschaut, ob hier immer noch ein executeable jar rauskommt? Ich hatte das mal, dass das Projekt ohne dem Plugin zwar in IntelliJ gestartet ist, aber das Manifest in der Pom nicht gepasst hat.
@@ -63,7 +59,7 @@ | |||
<plugin> | |||
<groupId>org.apache.maven.plugins</groupId> | |||
<artifactId>maven-compiler-plugin</artifactId> | |||
<version>${maven.compiler.plugin.version}</version> | |||
<version>3.11.0</version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Welchem Pattern folgst du hier, dass du manche versionen in die props rein nimmst und andere nicht?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alles was ich min. 2 definiere schreibe ich in die Properties, sonst direkt zum Plugin
Issue
Description
Checklist