search

openpharma / roxytypes

typed parameter definitions for `roxygen2`
https://openpharma.github.io/roxytypes/
Other
4 stars 1 forks source link

`convert()` would be nicer if it picked up the format from `DESCRIPTION` #7

Closed danielinteractive closed 1 year ago

danielinteractive commented 1 year ago

Currently, in my test case on jmpost repo I need to manually give the "old" manual format as argument to convert(), even though I specified it already in Config/roxytypes: list(format = "({type})\\cr {description}"). It would be nicer from convert() if it picked up the default "old" manual = target format from DESCRIPTION .

dgkf commented 1 year ago

Great idea! I hadn't even thought of that as a workflow. I always considered Config/roxytypes' format as an "output" and the convert argument as an "input", but if you want a 1-to-1 conversion, it makes sense that they would be one-in-the-same.

dgkf commented 1 year ago

Just FYI - Have reproduced it locally and will try to knock out a fix.