search

ComputationalProteomicsUnit / maker

Makefile for R packages
GNU General Public License v3.0
31 stars 5 forks source link

update-date #18

Closed lbraglia closed 9 years ago

lbraglia commented 9 years ago

A proposed target (name can be changed) for those that always forget to update package release date (yours truly in first line)

sgibb commented 9 years ago

Nice idea but do you think this is really necessary? The increment-version-* targets do this automatically. But maybe we should update the date in the release target as well ... What do you think?

lbraglia commented 9 years ago

oops... didn't saw it was already implemented in increment-version-* :) ...

Adding "...and update DESCRIPTION Date field" to increment-version-* documentation lines would be enough IMHO, eg. no need to change release.

Another point could be whether to add or not Date: in DESCRIPTION, if absent. I see no cons of adding by default, but may be wrong/"paternalistic"... In the case, docs for increment-version-* could be "...and set DESCRIPTION Date field"

Let me know, Luca

sgibb commented 9 years ago

The Date field is added if missing. I also updated the docs.

lbraglia commented 9 years ago

ok, thanks!