Skip to content
This repository has been archived by the owner on Jul 30, 2024. It is now read-only.

feat: make isal optional and installed only as an extra #7

Merged
merged 1 commit into from
Jan 4, 2024
Merged

Conversation

bdraco
Copy link
Owner

@bdraco bdraco commented Jan 4, 2024

We have problems detecting i686 during image builds because they are done on x86_64 machines. To avoid this issue isal is now an extra and needs to be installed as pip install aiohttp-zlib-ng[isal]

We have problems detecting i686 during image builds because they are
done on x86_64 machines. To avoid this issue isal is now an extra
and needs to be installed as `pip install aiohttp-zlib-ng[isal]`
Copy link

codecov bot commented Jan 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ecdaf7d) 71.42% compared to head (399e2e5) 80.95%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
+ Coverage   71.42%   80.95%   +9.52%     
==========================================
  Files           1        1              
  Lines          21       21              
  Branches        4        4              
==========================================
+ Hits           15       17       +2     
+ Misses          4        2       -2     
  Partials        2        2              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bdraco bdraco merged commit 0b5754b into main Jan 4, 2024
20 of 21 checks passed
@bdraco bdraco deleted the extras branch January 4, 2024 21:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant