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

Avoid importing unused cpython array for compatibility with pypy #109

Merged
merged 1 commit into from
Oct 12, 2020

Conversation

hmaarrfk
Copy link
Contributor

@hmaarrfk hmaarrfk commented Oct 12, 2020

Do you have a changelog you want me to add a note in?

A good explanation as to why this is important
h5py/h5py#1514

@lindstro
Copy link
Member

We keep the changelog locally. If you write a short note, I will include it with the next release.

Thanks for identifying this issue. I will merge once the tests pass.

@hmaarrfk
Copy link
Contributor Author

I think the title says it all. Thanks!

@codecov-io
Copy link

Codecov Report

Merging #109 into develop will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #109      +/-   ##
===========================================
- Coverage    94.40%   94.40%   -0.01%     
===========================================
  Files          263      263              
  Lines        12683    12682       -1     
===========================================
- Hits         11973    11972       -1     
  Misses         710      710              
Impacted Files Coverage Δ
python/zfpy.pyx 90.49% <ø> (-0.05%) ⬇️

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 50aa4f1...ec8a2d3. Read the comment docs.

@lindstro lindstro merged commit 75c2a59 into LLNL:develop Oct 12, 2020
@hmaarrfk hmaarrfk deleted the pypy_compat branch October 12, 2020 18:08
@GarrettDMorrison
Copy link
Member

LGTM

@hmaarrfk hmaarrfk mentioned this pull request Sep 1, 2021
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.

4 participants