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

Pull #3820: Rename method in CheckUtil #3820

Merged
merged 1 commit into from Feb 11, 2017

Conversation

Projects
None yet
3 participants
@Vladlis
Member

Vladlis commented Feb 11, 2017

The method returns token names, not text.

@codecov-io

This comment has been minimized.

Show comment
Hide comment
@codecov-io

codecov-io Feb 11, 2017

Codecov Report

Merging #3820 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #3820   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files         275     275           
  Lines       13618   13618           
  Branches     3066    3066           
======================================
  Hits        13618   13618

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 c9fc6ba...bf70946. Read the comment docs.

codecov-io commented Feb 11, 2017

Codecov Report

Merging #3820 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #3820   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files         275     275           
  Lines       13618   13618           
  Branches     3066    3066           
======================================
  Hits        13618   13618

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 c9fc6ba...bf70946. Read the comment docs.

@romani romani merged commit d3993fe into checkstyle:master Feb 11, 2017

6 of 7 checks passed

wercker/build Wercker pipeline failed
Details
IDEA Inspections Pull Request (Checkstyle) TeamCity build finished
Details
codacy/pr Good work! A positive pull request.
Details
codecov/patch Coverage not affected when comparing c9fc6ba...bf70946
Details
codecov/project 100% remains the same compared to c9fc6ba
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@Vladlis Vladlis deleted the Vladlis:wrong-method-name branch Feb 16, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment