Add objects attribute to Token - #644
Conversation
|
@flaeppe one more question. I really hope that this change won't affect user-defined models. (I think we would have lots of tests failing then, but still. Since many models in the wild don't have explicit managers defined). |
|
I'm not sure, was thinking about bisecting from #643 and see what commit I run into |
|
Oh, right, it was green 11 hours ago in https://github.com/typeddjango/djangorestframework-stubs/actions/runs/10099849613
|
|
I've double checked, it's indeed from typeddjango/django-stubs#2276 It looks like a repro case should be trivial in this case? Gonna see if I can figure it out |
|
I don't have a repro other than this, but I think I figured it out: The thing is that since typeddjango/django-stubs#2276 now goes through the mro it conflicts with the |
|
I'm leaning towards reverting the plugin code that removes |
|
@sobolevn I've opened typeddjango/django-stubs#2280 |
|
This PR is still fine, because it will also help |
Yes, I remember now that adding explicit
objectshelps withpyright