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 nullability annotation in Subject #891

Merged
merged 1 commit into from
Nov 16, 2022

Conversation

atifaziz
Copy link
Member

This PR fixes the nullability annotation in the private helper method of Subject so a null-forgiving operator is not needed at the call site.

@atifaziz atifaziz self-assigned this Nov 16, 2022
@codecov
Copy link

codecov bot commented Nov 16, 2022

Codecov Report

Merging #891 (d5f066c) into master (c6ca90d) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #891   +/-   ##
=======================================
  Coverage   92.53%   92.53%           
=======================================
  Files         108      108           
  Lines        3428     3428           
  Branches     1020     1020           
=======================================
  Hits         3172     3172           
  Misses        194      194           
  Partials       62       62           
Impacted Files Coverage Δ
MoreLinq/Reactive/Subject.cs 81.53% <0.00%> (ø)

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

@atifaziz atifaziz merged commit d587416 into morelinq:master Nov 16, 2022
@atifaziz atifaziz deleted the fix-nrt-subject branch November 16, 2022 19:54
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.

1 participant