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

4 participants