Skip to content

Conversation

@2bndy5
Copy link
Collaborator

@2bndy5 2bndy5 commented Aug 19, 2022

The README states that the value of true focuses on lines in diff with additions only.
Setting it to 'diff' will use the original behavior (meaning entire diff chunks with unchanged lines).

However, I forgot to change this value's interpretation in the src code (& test).

This commit makes the README accurate.

The README states that the value of true focuses on
lines in diff with additions only.
Setting it to 'diff' will use the original behavior (meaning
entire diff chunks with unchanged lines).

However, I forgot to change this value's
interpretation in the src code (& test).

This commit makes the README accurate.
@2bndy5 2bndy5 added the bug Something isn't working label Aug 19, 2022
@2bndy5 2bndy5 requested a review from shenxianpeng August 19, 2022 11:07
@codecov-commenter
Copy link

codecov-commenter commented Aug 19, 2022

Codecov Report

Merging #90 (a03b252) into master (600b1aa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #90   +/-   ##
=======================================
  Coverage   80.47%   80.47%           
=======================================
  Files           6        6           
  Lines         630      630           
=======================================
  Hits          507      507           
  Misses        123      123           
Impacted Files Coverage Δ
cpp_linter/run.py 78.76% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@2bndy5
Copy link
Collaborator Author

2bndy5 commented Aug 19, 2022

Related to #74 (comment)

@2bndy5 2bndy5 merged commit 92298bd into master Aug 19, 2022
@2bndy5 2bndy5 deleted the fix-setting-lines-changed-only branch August 19, 2022 18:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants