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

Mark local function static where possible #922

Merged
merged 1 commit into from
Jan 15, 2023

Conversation

atifaziz
Copy link
Member

This PR marks local functions as static, those that don't access variables from their enclosing scope.

@codecov
Copy link

codecov bot commented Jan 15, 2023

Codecov Report

Merging #922 (63d4efe) into master (78b8699) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #922   +/-   ##
=======================================
  Coverage   92.44%   92.44%           
=======================================
  Files         112      112           
  Lines        3440     3440           
  Branches     1018     1019    +1     
=======================================
  Hits         3180     3180           
  Misses        199      199           
  Partials       61       61           
Impacted Files Coverage Δ
MoreLinq/ToDataTable.cs 96.87% <ø> (ø)

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

@atifaziz atifaziz merged commit 63d4efe into morelinq:master Jan 15, 2023
@atifaziz atifaziz deleted the local-fn-static branch January 15, 2023 12: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.

2 participants