Introduce RemovedInDRF…Warning classes to simplify deprecations. by carltongibson · Pull Request #6480 · encode/django-rest-framework (original) (raw)

Hi @auvipy.

To be honest I too find it distracting that you approve PRs without adding to the conversation.

For me it would be better if you spent your time looking at less PRs but reviewed them fully, with comments and improvements etc. There’s a massive backlog around the ecosystem so plenty to do there.

Then if there were some PRs that you just approved that would tell something. As it stands it your tick isn’t often acting as a marker, since it’s too quickly given, so it’s not serving as a guide as to whether to merge.

I hope that makes sense. It’s meant in a spirit of love and encouragement. Please think about it. Happy to chat off thread if you like. C.