The column callback module shouldn't pass objects that are Ansible specific. For example:
run_on_playbook_stats passes a stats parameter that is an instance of AggregateStats. This class is Ansible defined and has strong possibly to change its methods or interface.
The same goes for the playbook, task, play parameters which has unknown type. These need to be docstring'd also so a user can know.
The api_runner.run_playbook() already returns a stats dict that has been processed using _process_stats(). The same needs to be done for the callback.
The column callback module shouldn't pass objects that are Ansible specific. For example:
run_on_playbook_stats passes a stats parameter that is an instance of AggregateStats. This class is Ansible defined and has strong possibly to change its methods or interface.
The same goes for the playbook, task, play parameters which has unknown type. These need to be docstring'd also so a user can know.
The api_runner.run_playbook() already returns a stats dict that has been processed using _process_stats(). The same needs to be done for the callback.