[R-package] remove unused '...' in Booster constructor

[jameslamb]

Contributes to #4310 (based on #4226 (comment)).

This proposes removing the ... from the constructor of the Booster class in the R package. That class is not exported from the R package, so its constructor can be considered a private API and this change should be considered non-breaking (similar to #4338).

Removing this use of ... makes the package's code slightly stricter, which reduces the risk of bugs.

Notes for Reviewers

Internal calls of the Booster class always use that class's existing keyword arguments.

LightGBM/R-package/R/lgb.Booster.R

Line 812 in 86ead20

return(invisible(Booster$new(modelfile = filename)))

LightGBM/R-package/R/lgb.Booster.R

Line 819 in 86ead20

return(invisible(Booster$new(model_str = model_str)))

LightGBM/R-package/R/lgb.train.R

Line 264 in 86ead20

booster <- Booster$new(params = params, train_set = data)

LightGBM/R-package/R/lgb.cv.R

Line 341 in 86ead20

booster <- Booster$new(params = params, train_set = dtrain)

Found with git grep -E 'Booster\$new'.

[StrikerRUS]

LGTM!

[github-actions]

This pull request has been automatically locked since there has not been any recent activity since it was closed. To start a new related discussion, open a new issue at https://github.com/microsoft/LightGBM/issues including a reference to this.