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

Issue #58: added some code coverage for Main #92

Merged
merged 1 commit into from
Nov 15, 2017

Conversation

rnveach
Copy link
Member

@rnveach rnveach commented Nov 15, 2017

Issue #58

Main's main method was changed to String... to make it easier to pass arguments.
Exception exit was changed to -1 to mimic checkstyle when error occurs.

Code coverage is raised to 75% for this class.

@rnveach
Copy link
Member Author

rnveach commented Nov 15, 2017

Same issue as other PR. If problem continues, I'll try to find a new commit.

@rnveach rnveach merged commit 6831002 into checkstyle:master Nov 15, 2017
@rnveach rnveach deleted the issue_58 branch November 15, 2017 02:18
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.

1 participant