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

Issue #200: Edit default config for checkstyle tester to produce no violations #201

Merged
merged 1 commit into from
Mar 27, 2017
Merged

Issue #200: Edit default config for checkstyle tester to produce no violations #201

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
4 changes: 2 additions & 2 deletions checkstyle-tester/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ The script receives the following command line arguments:

**listOfProjects** (l) - path to the file which contains the projects which sources will be analyzed by Checkstyle during report generation (required);

**config** (c) - path to the file with Checkstyle configuration (required);
**config** (c) - path to the file with Checkstyle configuration (required). The default config should be changed in order to be appropriate for your use purposes;

**ignoreExceptions** (i) - whether Checkstyle Maven Plugin should ignore exceptions (optional, default is false).

Expand Down Expand Up @@ -130,7 +130,7 @@ The script receives the following set of command line arguments:

**patchConfig** (pc) - path to the patch checkstyle config file. It will be applied to patch branch (required if baseConfig is specified);

**config** (c) - path to the checkstyle config file. It will be applied to base and patch branches (required if baseConfig and patchConfig are not secified);
**config** (c) - path to the checkstyle config file. It will be applied to base and patch branches (required if baseConfig and patchConfig are not secified). The default config should be changed in order to be appropriate for your use purposes;

**listOfProjects** (l) - path to the file which contains the projects which sources will be analyzed by Checkstyle during report generation.

Expand Down
8 changes: 5 additions & 3 deletions checkstyle-tester/my_check.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

<module name = "Checker">
<property name="charset" value="UTF-8"/>

<!-- do not change severity to 'error', as that will hide errors caused by exceptions -->
<property name="severity" value="warning"/>

Expand All @@ -21,9 +21,11 @@
<module name="TreeWalker">
<!-- Example of sevntu.checkstyle Check usage -->
<!-- <module name="NestedSwitchCheck"/> -->

<!-- Example of checkstyle Check usage -->
<module name="AbstractClassName"/>
<module name="ThrowsCount">
<property name="max" value="20000000"/>
</module>
</module>
<!--
<module name="SeverityMatchFilter">
Expand Down