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 #4876: specified what checks is in ModuleReflectionUtils #5326

Merged
merged 1 commit into from Dec 8, 2017

Conversation

rnveach
Copy link
Member

@rnveach rnveach commented Dec 5, 2017

Issue #4876

Used method name talked about in issue. If you can think of a better one, let me know.

@rnveach rnveach requested a review from romani December 5, 2017 23:27
@codecov-io
Copy link

codecov-io commented Dec 6, 2017

Codecov Report

Merging #5326 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #5326   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files         296     296           
  Lines       16160   16160           
  Branches     3686    3686           
======================================
  Hits        16160   16160
Impacted Files Coverage Δ
.../tools/checkstyle/utils/ModuleReflectionUtils.java 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 799b25c...25b2f36. Read the comment docs.

@romani romani merged commit 2e3ff43 into checkstyle:master Dec 8, 2017
@rnveach rnveach deleted the issue_4876 branch December 8, 2017 14:53
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.

None yet

3 participants