meeb / django-distill

Minimal configuration static site generator for Django
MIT License
441 stars 35 forks source link

allow substituting a custom DistillRender class via settings #67

Closed anentropic closed 1 year ago

anentropic commented 1 year ago

As a less obtrusive change than https://github.com/meeb/django-distill/pull/66 ...if I could substitute a custom renderer class then I could implement the changes from the other PR just as a user of the lib.

But maybe this is a complementary feature that is also nice to have.

meeb commented 1 year ago

I have no objection to adding this. Do you actually have a custom renderer class or is this just future proofing?

anentropic commented 1 year ago

I have no objection to adding this. Do you actually have a custom renderer class or is this just future proofing?

I don't at the moment, the one I would have made is basically what's in the other PR

meeb commented 1 year ago

Fair enough! Seems fine to me.