Why is Graphviz being installed?

[akademy]

Why is graphviz defined in the template? Seems like this will be unnecessary in most pips.

python-copier-template/template/Dockerfile.jinja

Line 8 in 2d76110

graphviz \

It's probably being installed by many projects for no reason.

[coretl]

It's there because the default api docs used to include a class hierarchy which required graphviz, however although the directive is included:

python-copier-template/template/{% if sphinx %}docs{% endif %}/conf.py.jinja

Line 45 in 2d76110

"sphinx.ext.inheritance_diagram",

It doesn't appear to be used in the docs at all. Maybe we should remove both the directive and the apt install and make a how-to telling you how to put it back in?

[akademy]

Yeah, sounds sensible (can always put it back again if it causes a problem).

I guess we'll still need to include the apt-get update though, as some builds may assume it's already run? (Or perhaps no‑one retrospectively copies the template...)