GitHub user ham1 opened a pull request:

    https://github.com/apache/jmeter/pull/365

    Expanded Checkstyle to files in src and test; fixed newly checked files

    ## Description
    Added `test` to the files being checked.
    
    ## Motivation and Context
    Felix and Philippe found some groovy files without the headers, this fixes 
Checkstyle not picking that up.
    
    ## How Has This Been Tested?
    `and checkstyle`
    
    ## Checklist:
    <!--- Go over all the following points, and put an `x` in all the boxes 
that apply. -->
    <!--- If you're unsure about any of these, don't hesitate to ask. We're 
here to help! -->
    - [x] My code follows the [code style][style-guide] of this project.
    - [x] I have updated the documentation accordingly.
    
    [style-guide]: https://wiki.apache.org/jmeter/CodeStyleGuidelines


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ham1/jmeter increase_checkstyle_file_coverage

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/jmeter/pull/365.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #365
    
----
commit 6f2c28508b0f3b0f1873a1702d5867aaf8790e15
Author: Graham Russell <graham@...>
Date:   2017-12-20T14:50:21Z

    Expanded Checkstyle to files in src and test; fixed newly checked files

----


---

Reply via email to