-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/61937/#review183915
-----------------------------------------------------------
```<exclude>**/.*</exclude>``` looks like overkill, and also in my opinion
node_modules and dist should be excluded in order to make local builds pass, so
that's what works for me:
```
<excludes>
<exclude>**/README.md</exclude>
<exclude>**/*.json</exclude>
<exclude>**/*.log</exclude>
<exclude>**/*.txt</exclude>
<exclude>**/*.story</exclude>
<exclude>**/*.editorconfig</exclude>
<exclude>**/src/vendor/**</exclude>
<exclude>**/yarn.lock</exclude>
<exclude>**/docker/Profile</exclude>
<exclude>**/node_modules/**</exclude>
<exclude>**/dist/**</exclude>
</excludes>
```
- Andrii Babiichuk
On Сер. 28, 2017, 11:16 до полудня, Oliver Szabo wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/61937/
> -----------------------------------------------------------
>
> (Updated Сер. 28, 2017, 11:16 до полудня)
>
>
> Review request for Ambari, Andrii Babiichuk and Miklos Gergely.
>
>
> Bugs: AMBARI-21825
> https://issues.apache.org/jira/browse/AMBARI-21825
>
>
> Repository: ambari
>
>
> Description
> -------
>
> exclude some files from rat checks: vendor files, .* and README.md files
>
>
> (note: by the way, i do not know the .editorconfig file is required for us or
> not)
>
>
> Diffs
> -----
>
> ambari-logsearch/ambari-logsearch-web-new/README.md 2fb07fb
> ambari-logsearch/pom.xml 2be11ee
>
>
> Diff: https://reviews.apache.org/r/61937/diff/1/
>
>
> Testing
> -------
>
> ran mvn clean package successfully
>
>
> Thanks,
>
> Oliver Szabo
>
>
