Requested Changes for reformat_yaml_data and add_check_to_list

[Jacob-Allebach]

From Pull Request #54

Lines 21 and 33:
To resolve the issue with sending None to this function explicitly, here's what you could do: move the path parameter to the end of the parameter list, and then give it a default value of None. Something like:

Suggested change (Line 21):

- def add_checks_to_list(path, data_list, reformatted_data):
+ def add_checks_to_list(data_list, reformatted_data, path=None):

Then, you can call add_checks_to_list with no path parameter to specify starting at the root (this would replace where you're currently manually specifying None), or give it a path if you're starting an inner recursive call (where you're calling it inside this function).

(Note: Would also need to change line 33 to put the path at the end of the function call)

Lines 26-28 and 33-35:
To make this code look better after running black (i.e. not have the line splitting on the parameter list), you should place the comment above the if statement, instead of after it. This will result in cleaner-looking code!

Line 38:
Instead of modifying reformatted_data and passing it by reference to ensure the values are returned, you should directly create and return a new list in this function, rather than creating it in reformat_yaml_data. When processing the result of a recursive call, just use reformatted_data.extend to add on the recursively-generated checks to the current data.