suminb / hanja

한글, 한자 라이브러리
134 stars 16 forks source link

Lazily import implementations #14

Closed dahlia closed 3 years ago

dahlia commented 3 years ago

This fixes ImportError that setup.py throws:

$ python setup.py 
Traceback (most recent call last):
  File "setup.py", line 6, in <module>
    import hanja
  File "hanja/__init__.py", line 8, in <module>
    from hanja.impl import is_hanja, is_valid_mode, split_hanja, translate
  File "hanja/impl.py", line 5, in <module>
    from hanja.table import hanja_table
  File "hanja/table.py", line 2, in <module>
    import yaml
ImportError: No module named yaml
google-cla[bot] commented 3 years ago

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

google-cla[bot] commented 3 years ago

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers

ℹ️ Googlers: Go here for more info.

dahlia commented 3 years ago

@googlebot I signed it!

coveralls commented 3 years ago

Coverage Status

Coverage increased (+0.7%) to 93.846% when pulling 5c1a3c237747a9c4f30cc35d8222b846c8d6dc66 on dahlia:make-setup.py-independent into e1ec45b48c0089ff7a80b7eacfb59bb225639957 on suminb:develop.

dahlia commented 3 years ago

A build job (pypy) failed because the cryptography package was failed to be installed. I believe the job fail is irrelevant to this pull request. Other build jobs all succeeded.