Add support to scan specific files only

Review Request #9530 — Created Jan. 25, 2018 and submitted

Information

ReviewBot
master
5befb92...

Reviewers

PMD supports multiple languages. But most checks
are for java files only as the other languages
are for CPD only or requires manually added checks.

This features allows to increase the performance
of review checks as it is not necessary to run PMD
for all files in a diff if only .java has enabled
checks. Also this avoid but not fix bug #4633.

Also increase timeout as big patches with a lot
of files will take some time to be finished.

Fixes conflicting variable name 'f' introduced by
changeset f95629e2739037bd0e0f289875f710f2da4aa2dd
by mistake.

Added some binary files with another file extension
and saw that it wasn't scanned.

Enabled java extension only and added a java file.
Saw that the check found correct issues.

Leave the extension setting empty and saw that it
still found correct issues in uploaded java patch.

Used "cpp" as file extension and saw that it
ignores java files.

Description From Last Updated

Can you flesh out the description a bit? The summary "Add support to scan specific files only" doesn't adequately describe …

daviddavid

We generally prefer putting the joining space at the end of the line instead of the beginning of the next.

daviddavid
misery
misery
david
  1. 
      
  2. Show all issues

    Can you flesh out the description a bit? The summary "Add support to scan specific files only" doesn't adequately describe what the change does or the motivation for it.

    See https://www.notion.so/reviewboard/Writing-Good-Change-Descriptions-10529e7c207743fa8ca90153d4b21fea for our guidelines on this.

  3. bot/reviewbot/tools/pmd.py (Diff revision 1)
     
     
     
    Show all issues

    We generally prefer putting the joining space at the end of the line instead of the beginning of the next.

  4. 
      
misery
misery
david
  1. Ship It!
  2. 
      
misery
Review request changed
Status:
Completed
Change Summary:
Pushed to master (7902c2d)