adamMaier / reviewr

Assists the review of data wrangling and anlaysis code
6 stars 2 forks source link

Provide options for levels of verbosity? #4

Open MatthieuStigler opened 5 years ago

MatthieuStigler commented 5 years ago

Nice package!

Just discovered it, and I like the join_qc function!! I am wondering whether you considered having verbosity options, at the package level? Typically, doing a left_join and having a message telling me it's a one to many join is not very relevant to me. Likewise, if there are 0 additional rows, I don't need necessary that info to be printed . Would having a system of verbosity levels, and letting user change default make sense?

Thanks!

adamMaier commented 5 years ago

Thanks for the nice words and suggestion. I think that's a nice idea and something I can implement. As you can see from the commit history, my work on it goes in fits, but I'll try to add that as soon as possible.