ropensci / piggyback

:package: for using large(r) data files on GitHub
https://docs.ropensci.org/piggyback
GNU General Public License v3.0
182 stars 26 forks source link

overwrite logic is confusing #25

Closed cboettig closed 5 years ago

cboettig commented 5 years ago

Having separate options use_timestamps = T/F and overwrite = T/F is confusing. This would be much clearer if overwrite simply took one of three values: TRUE, FALSE, or use_timestamps, since the latter is really an intermediate overwrite rule (i.e. always overwrite, never overwrite, or overwrite only older timestamps).

Of course this means mixing logical and character input, but I guess that's okay.