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

fix setting of lines-changed-only #90

Merged
merged 1 commit into from
Aug 19, 2022
Merged

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.

3 participants