the `py.test workaround hides legitimate mistakes

pytest-dev / apipkg

MIT License

55 stars 16 forks source link

the `py.test workaround hides legitimate mistakes #10

Closed RonnyPfannschmidt closed 3 years ago

RonnyPfannschmidt commented 6 years ago

c9b997713cb77d2c1334acb7847ee6b24e3261b2 introduced returning None on missing aliasmodules unconditionally this does hide actual errors

SimonBiggs commented 4 years ago

Thank you for this bug report, this helped highlight to me that I can make apipkg workable for me by using apipkg >=1.0,<=1.2. That way error messages for missing imports are given to the user as an ImportError instead of None not callable or something of the like.

Given that Python 3.4 has now reached its end of life (https://www.python.org/dev/peps/pep-0429/#id4) I am of the opinion that this workaround should now be reverted.