Skip to content

add-some-unit-tests#19

Closed
TheRealHaui wants to merge 1 commit into
apache:masterfrom
TheRealHaui:add-some-unit-tests
Closed

add-some-unit-tests#19
TheRealHaui wants to merge 1 commit into
apache:masterfrom
TheRealHaui:add-some-unit-tests

Conversation

@TheRealHaui

Copy link
Copy Markdown

I have written some Unit Tests that increase code coverage and want to contribute them.

@coveralls

coveralls commented Jul 3, 2017

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.0%) to 94.337% when pulling ee06d3a on TheRealHaui:add-some-unit-tests into 0bdca58 on apache:master.

1 similar comment
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.0%) to 94.337% when pulling ee06d3a on TheRealHaui:add-some-unit-tests into 0bdca58 on apache:master.

@TheRealHaui

Copy link
Copy Markdown
Author

@garydgregory
Could you be so kind an merge my contribution?

import java.nio.charset.Charset;
import java.nio.charset.StandardCharsets;
import java.sql.*;
import java.util.*;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No star imports here please.

@asfgit asfgit closed this in e41d484 Jul 10, 2017
@garydgregory

Copy link
Copy Markdown
Member

I had brought this changes into my local workspace last week and fiddled with them, but not pushed out the changed. Committed now, with some changes.

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