search

baumer-lab / fertile

creating optimal conditions for reproducibility
GNU General Public License v3.0
52 stars 4 forks source link

rename check()? #61

Closed beanumber closed 4 years ago

beanumber commented 4 years ago

@jennybc says:

The name collision between fertile::check() and devtools::check() gives me pause. I guess it could be seen as a positive? But I think it's more negative.

How about fertile::proj_check()?

jennybc commented 4 years ago

Is there an existing naming pattern already, e.g. a prefix?

ambertin commented 4 years ago

@jennybc The proj_ prefix is used quite regularly throughout the package, including in the other major function from the package (proj_test()) and its component functions. Renaming fertile::check() to fertile::proj_check(), as suggested by @beanumber, seems like a good way to go in order to maintain consistency!