You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
#633 has continued to accumulate additional reports after being closed, so I'm opening a new issue here.
As others have said, @inheritDotParams currently (r-lib/pkgdown@de95b13) omits ... from the list of inherited arguments when the function being inherited from has its own ... argument.
Then, users of api() would be able to immediately tell that ... arguments passed to api() flow down to core() if they are not intercepted by middle().
Alternatively, inherited parameters from the whole function hierarchy could be listed directly in api.Rd, but that might get too messy, as discussed in #633 (comment).
I realize that there's a workaround using two @inheritDotParams tags:
However, this workaround is a bit brittle; if I later change middle() to now call core2() instead of core(), then using this workaround, I have to go back and update the @inheritDotParams tags in bothmiddle() and api().
The text was updated successfully, but these errors were encountered:
#633 has continued to accumulate additional reports after being closed, so I'm opening a new issue here.
As others have said, @inheritDotParams currently (r-lib/pkgdown@de95b13) omits
...
from the list of inherited arguments when the function being inherited from has its own...
argument.Here's a reprex, modifying the one in #633 (comment):
pkgdown writes
api.Rd
aswhich renders as:
Instead, I'd like api.Rd to be:
with corresponding render:
Then, users of
api()
would be able to immediately tell that...
arguments passed toapi()
flow down tocore()
if they are not intercepted bymiddle()
.Alternatively, inherited parameters from the whole function hierarchy could be listed directly in
api.Rd
, but that might get too messy, as discussed in #633 (comment).I realize that there's a workaround using two
@inheritDotParams
tags:However, this workaround is a bit brittle; if I later change
middle()
to now callcore2()
instead ofcore()
, then using this workaround, I have to go back and update the@inheritDotParams
tags in bothmiddle()
andapi()
.The text was updated successfully, but these errors were encountered: