Add strict option for functions about broom.helpers HOT 2 CLOSED

I see. It could a good option and strict is I think the good argument name.
However, it should be generalized to each tidy_* functions.

• tidy_identify_variables(): check if variable names has been found. If it is not the case, var_type will be "unknown".
• tidy_add_contrasts(): check, if there are categorical variables only, that a contrasts value has been documented. Do we have a case where it will fail?
• tidy_add_reference_rows(): all categorical variables with a treatment, SAS or sum contrasts should have a reference row. Not sure if there is a case where it will fail.

and so on....

A difficulty is to identify cases where it will fail to develop the appropriate tests.

Not sure to have enough time this week but if you want to explore and to propose a PR, you are welcome

from broom.helpers.

I like adding the strict argument for tidy_identify_variables() and tidy_add_header_rows(show_single_row=) because the user is specifically asking for something that can result in an error.

For tidy_add_contrasts() I would only make it an error if there was a categorical variable that should have a contrast and it cannot be addressed.

For tidy_add_reference_rows() I would only make it an error if there was a categorical variable that should have a reference row that we cannot add.

I'll work on a PR.

from broom.helpers.