Skip to content
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.

Sign up for GitHub

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

Jump to bottom

profile added #13

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

profile added #13

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a0b3050
profile added
suvmanda Oct 7, 2020
c49e3bd
PR Comments Added
suvmanda Oct 7, 2020
f6717ae
PR comment fixed
suvmanda Oct 7, 2020
a80dfb7
PR Comments fixed
suvmanda Oct 7, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 0 additions & 5 deletions pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -343,9 +343,6 @@
<warName>${project.artifactId}</warName>
</configuration>
</plugin>

Copy link
Member

Choose a reason for hiding this comment

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

I meant that tags on the second level should have one line between them, tags on every other level should not. So if you coud add one empty line between 345-346 that would be great



<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-site-plugin</artifactId>
Expand Down Expand Up @@ -423,7 +420,6 @@
</plugin>
</plugins>
</reporting>

Copy link
Member

Choose a reason for hiding this comment

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

See review comment above, one empty line here between </reporting> and <profiles> would be perfect.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @lmarniazman . All comments are fixed.

<profiles>
<!-- JaCoCo for test coverage -->
Copy link
Member

Choose a reason for hiding this comment

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

@hohwille Was this how you meant to include the jacoco-report profile into maven-parent? I thought this was a good way of doing it. After this PR is merged, we can also delete the profile from the pom.xml of sonar-devon4j-plugin.

<profile>
Expand Down Expand Up @@ -451,7 +447,6 @@
</plugins>
</build>
</profile>

<profile>
<id>deploy</id>
<build>
Expand Down