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 compatibility wity beartype==0.9.1 #2633

Merged
merged 1 commit into from
Dec 23, 2021
Merged

Conversation

berland
Copy link
Contributor

@berland berland commented Dec 22, 2021

Issue
Resolves #2632

Approach
Follow up breaking change from beartype 0.8.1 to 0.9.x. Bump beartype requirements in setup.py. Komodo bleeding already has version 0.9.1

Pre review checklist

  • Added appropriate labels

@berland berland added the maintenance Not a bug now but could be one day, repaying technical debt label Dec 22, 2021
@codecov-commenter
Copy link

codecov-commenter commented Dec 22, 2021

Codecov Report

Merging #2633 (ffcbb12) into main (ebbca9c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2633   +/-   ##
=======================================
  Coverage   64.82%   64.82%           
=======================================
  Files         648      648           
  Lines       54188    54188           
  Branches     4517     4517           
=======================================
+ Hits        35128    35129    +1     
  Misses      17653    17653           
+ Partials     1407     1406    -1     
Impacted Files Coverage Δ
ert/data/record/_record.py 92.89% <100.00%> (ø)
libres/lib/enkf/block_fs_driver.cpp 81.92% <0.00%> (+0.56%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ebbca9c...ffcbb12. Read the comment docs.

setup.py Show resolved Hide resolved
@xjules
Copy link
Contributor

xjules commented Dec 22, 2021

The error in type-checking is caused by this it seems: python/mypy#9951

Needed to add upper limit in mypy
Copy link
Contributor

@dafeda dafeda left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@berland berland merged commit 5113f75 into equinor:main Dec 23, 2021
@berland berland deleted the beartype-0.9.1 branch January 13, 2022 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Not a bug now but could be one day, repaying technical debt
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Beartype Import Error in record
5 participants