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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

MaxLineLength: Add tests covering TWiStErRob's last example. #5711

Merged
merged 1 commit into from Jan 20, 2023

Conversation

TWiStErRob
Copy link
Member

@TWiStErRob TWiStErRob commented Jan 19, 2023

Followup on #5583 to cover #5314 (comment)

I ran the test before your fix in #5583, and they were misaligned as described in the comment, and after, they just pass as you said you tested 馃憦. This just codifies that verification.

@codecov
Copy link

codecov bot commented Jan 19, 2023

Codecov Report

Merging #5711 (9aa3041) into main (adf5d96) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #5711   +/-   ##
=========================================
  Coverage     84.46%   84.46%           
  Complexity     3733     3733           
=========================================
  Files           543      543           
  Lines         12705    12705           
  Branches       2222     2222           
=========================================
  Hits          10731    10731           
  Misses          865      865           
  Partials       1109     1109           

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

@BraisGabin BraisGabin merged commit 425df93 into main Jan 20, 2023
@BraisGabin BraisGabin deleted the followup-5583-tests branch January 20, 2023 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants