Skip to content

Commit

Permalink
PR remarks fixed
Browse files Browse the repository at this point in the history
  • Loading branch information
alevohin committed Jan 14, 2015
1 parent d3e55bf commit 1caea2b
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
1 change: 1 addition & 0 deletions qulice-maven-plugin/src/it/findbugs-exclude/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@
<excludes>
<!-- @todo #352 Need to add multiline check after issue #350-->
<!-- resolved. Now only one class can be excluded -->
<!-- Don't forget update example-exclude.apt.vm -->
<exclude>findbugs:~com.qulice.foo.*</exclude>
</excludes>
</configuration>
Expand Down
6 changes: 3 additions & 3 deletions qulice-maven-plugin/src/site/apt/example-exclude.apt.vm
Original file line number Diff line number Diff line change
Expand Up @@ -72,9 +72,9 @@ Exclude
2) <<<codenarc:**/src/ex1/Main.groovy,**/src/ex2/Main2.groovy>>> excludes
<<<Main.groovy>>> and <<<Main2.groovy>>> in appropriate packages.

Exclude for Findbug is a bit different, as expression must be exact class name
or starts with the ~ character the rest of content is interpreted as
a Java regular expression. For example:
Exclude for Findbug is a bit different, as expression has to be either an
exact class name or start with the ~ character the rest of content is
interpreted as a Java regular expression. For example:
1) <<<findbugs:~com.qulice.foo.*>>> excludes all classes in package
com.qulice.foo;
2) <<<findbugs:com.qulice.foo.Bar>>> excludes only class Bar in appropriate
Expand Down

0 comments on commit 1caea2b

Please sign in to comment.