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.

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

Added file extensions configuration for checkstyle #100

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tasmdis
Copy link

@tasmdis tasmdis commented Jun 9, 2015

Checkstyle is able to validate any type of file (not only Java), for example using RegexpSingleline module you can check xml or jsp files with specified regex pattern (trailing spaces). Before this change checkstyle processor checks only java files now user can specify supported file types by configure checkstyle.file.extensions property. Default value should be java, but you can easily change it to java, jsp,xml .

@codecov-io
Copy link

Current coverage is 80.53%

Merging #100 into master will change coverage by +1.27% by 074b09a

Powered by Codecov

@SpOOnman
Copy link
Collaborator

Seems fine, please add a unit test for that.

@SpOOnman
Copy link
Collaborator

Can you update this PR @tasmdis ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants