Message315020
While I like the idea of this change, the "partially initialized" addition is fairly subtle, and relatively easy to miss.
Perhaps append "(most likely due to a circular import)" to the partially initialized case?:
AttributeError: partially initialized "module 'spam' has no attribute 'ham' (most likely due to a circular import).
Crucially, for folks encountering the error for the first time, that also introduces them to the main phrase they may want to search for: "circular import".
The "most likely" weasel wording stems from the fact that the problem won't always be with the circular import - they may have just straight up referenced the wrong module or the wrong attribute name, so the apparently circular import is an error. |
|
Date |
User |
Action |
Args |
2018-04-06 15:57:44 | ncoghlan | set | recipients:
+ ncoghlan, brett.cannon, eric.snow, serhiy.storchaka |
2018-04-06 15:57:44 | ncoghlan | set | messageid: <1523030264.92.0.682650639539.issue33237@psf.upfronthosting.co.za> |
2018-04-06 15:57:44 | ncoghlan | link | issue33237 messages |
2018-04-06 15:57:44 | ncoghlan | create | |
|