Skip to content

Conversation

@garrettwrong
Copy link
Collaborator

@garrettwrong garrettwrong commented Jul 7, 2022

Apparently this is what they are concerned about...

https://docs.python-guide.org/writing/gotchas/#late-binding-closures

Hopefully after the patch we still pass pytest 😅 .

@garrettwrong garrettwrong added CI Continuous Integration cleanup labels Jul 7, 2022
@garrettwrong garrettwrong self-assigned this Jul 7, 2022
@garrettwrong garrettwrong linked an issue Jul 7, 2022 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Jul 7, 2022

Codecov Report

Merging #653 (8cd54f7) into develop (a4deec8) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #653   +/-   ##
========================================
  Coverage    86.50%   86.50%           
========================================
  Files          109      109           
  Lines         8293     8293           
========================================
  Hits          7174     7174           
  Misses        1119     1119           
Impacted Files Coverage Δ
src/aspire/covariance/covar2d.py 94.21% <100.00%> (ø)

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@garrettwrong garrettwrong marked this pull request as ready for review July 7, 2022 01:08
@garrettwrong garrettwrong requested a review from janden as a code owner July 7, 2022 01:08
@garrettwrong
Copy link
Collaborator Author

@chris-langfield , any concerns with this? I'd like to merge it in. We can skip Joakim on it if you have time to check it... Thanks

@garrettwrong garrettwrong merged commit e7d89ef into ComputationalCryoEM:develop Jul 7, 2022
@garrettwrong garrettwrong deleted the fixup_flakes branch July 7, 2022 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous Integration cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Recent CI Flake8 failure

2 participants