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

Make use of message in Debug.Assert #959

Merged
merged 1 commit into from
Jan 30, 2023
Merged

Conversation

i3arnon
Copy link
Contributor

@i3arnon i3arnon commented Jan 30, 2023

I assume the argument expression in message was originally meant to be used.

I assume the argument expression in `message` was originally meant to be used.
@codecov
Copy link

codecov bot commented Jan 30, 2023

Codecov Report

Merging #959 (cbb8a95) into master (283c44c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #959   +/-   ##
=======================================
  Coverage   92.52%   92.52%           
=======================================
  Files         113      113           
  Lines        3440     3440           
  Branches     1024     1024           
=======================================
  Hits         3183     3183           
  Misses        192      192           
  Partials       65       65           
Impacted Files Coverage Δ
MoreLinq/Debug.cs 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@atifaziz atifaziz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this oversight!

@atifaziz atifaziz merged commit 284b5e3 into morelinq:master Jan 30, 2023
@i3arnon i3arnon deleted the patch-2 branch January 30, 2023 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants