Reverse dependency issue related to how `fill()` is re-exported

[DavisVaughan]

Hi, I see you reexport fill() from tidyr here and also add your own method:

bupaR/R/fill.R

Lines 1 to 8 in c9b9eed

	#' @title Fill event log
	#' @param data \code{\link{log}}: Object of class \code{\link{eventlog}} or \code{\link{activitylog}}.
	#' @param ... Additional arguments passed to [tidyr][fill]
	#' @inheritParams tidyr::fill
	#' @name fill
	#' @importFrom tidyr fill
	#' @export
	tidyr::fill

Unfortunately the way you re-export is incorrect:

#' @title Fill event log
#' @param data \code{\link{log}}: Object of class \code{\link{eventlog}} or \code{\link{activitylog}}.
#' @param ... Additional arguments passed to [tidyr][fill]
#' @inheritParams tidyr::fill
#' @name fill
#' @importFrom tidyr fill
#' @export
tidyr::fill

We have added a new .by argument to fill() in dev tidyr, and this causes your package to fail with:

Package: bupaR
Check: for code/documentation mismatches
New result: WARNING
  Codoc mismatches from Rd file 'fill.Rd':
  fill
    Code: function(data, ..., .by = NULL, .direction = c("down", "up",
                   "downup", "updown"))
    Docs: function(data, ..., .direction = c("down", "up", "downup",
                   "updown"))
    Argument names in code not in docs:
      .by
    Mismatches in argument names:
      Position: 3 Code: .by Docs: .direction

If you could please reduce this reexport down to the following, then it should work regardless of how we change fill()

#' @importFrom tidyr fille
#' @export
tidyr::fill

We are currently releasing tidyr, so you will likely get a note from CRAN about this soon.