search

achimnol / aiotools

Idiomatic asyncio utilties
https://aiotools.readthedocs.io
MIT License
154 stars 12 forks source link

fix: Import in macOS (regression from #47) #49

Closed achimnol closed 2 years ago

achimnol commented 2 years ago

Though aiotools.fork and aiotools.server modules do not support Windows, importing them is fine. The previously found error was due to an unused code: ctypes.CDLL(None).

This PR removes those unused codes to avoid confusion and restore the import statements, as #47 has broken imports in macOS.

codecov[bot] commented 2 years ago

Codecov Report

Merging #49 (8d674b7) into main (2c0f9d9) will decrease coverage by 0.16%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
- Coverage   78.66%   78.50%   -0.17%     
==========================================
  Files          16       16              
  Lines        1336     1326      -10     
==========================================
- Hits         1051     1041      -10     
  Misses        285      285
Impacted Files Coverage Δ
src/aiotools/fork.py 74.48% <ø> (-0.69%) :arrow_down:
src/aiotools/__init__.py 93.33% <100.00%> (-1.91%) :arrow_down:

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 2c0f9d9...8d674b7. Read the comment docs.