Thank you for your review Jeremy! On 28/03/2024 16:48, Jeremy Spewock wrote:
I might add some kind of block comment here as a separator that delimits that metadata modifiers start here. Something like what is written in scapy.py which creates sections for XML-RPC method vs ones that are run by the docker container. This isn't something strictly necessary, but it might help break things up and add a little more explanation.
<snip>
You could do the same thing here for mixins, but again, I'm not sure it's really necessary.
Yes, I agree that using block comments to delimit sections is a good idea. I wasn't sure if we had an established way of doing this, and looks like we do!
