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

Replace most lambdas in tests with nested function definitions #436

Merged
merged 1 commit into from
Sep 22, 2019

Conversation

aleju
Copy link
Owner

@aleju aleju commented Sep 22, 2019

This is related to #414

@codecov-io
Copy link

Codecov Report

Merging #436 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #436   +/-   ##
=======================================
  Coverage   96.07%   96.07%           
=======================================
  Files          34       34           
  Lines       10750    10750           
=======================================
  Hits        10327    10327           
  Misses        423      423

@aleju aleju merged commit 739ae68 into master Sep 22, 2019
@aleju aleju deleted the remove_lambdas branch September 22, 2019 13:44
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