flowerysong

Microsoft defined this field as binary data, so I don't really think it's reasonable to expect the module to do anything other than return it as binary data. What is...

This whole file could be cleaned up quite a bit. It's more efficient to loop over the items that meet your criteria instead of looping over everything, and it's more...

All commit messages must have a sign-off indicating that you have reviewed the DCO (https://github.com/Klipper3d/klipper/blob/master/docs/developer-certificate-of-origin) and this commit is covered by one of its clauses. The process for doing this...

This is an unfortunate choice of return structure on the part of the module; `results` normally means that this is a nested set of results from a loop (https://docs.ansible.com/ansible/latest/playbook_guide/playbooks_loops.html#registering-variables-with-a-loop) but...

A quick hack: ```diff diff --git a/ara/ui/templates/result.html b/ara/ui/templates/result.html index c2eac09..8bca8f5 100644 --- a/ara/ui/templates/result.html +++ b/ara/ui/templates/result.html @@ -103,7 +103,7 @@ {% if field == "diff" %} {{ value | diff_result |...

Anything we do is going to be hacky, because ([unlike `prepared`](https://github.com/ansible/ansible/blob/b639bd1fd42917fceb1487ec2b5214df4db272c6/lib/ansible/plugins/callback/__init__.py#L342-L400)) this is not an `ansible-core` convention, but a collision between engine choices (loop results are put in `results`, `results`...