don't attempt to attachNamespace() twice

mlr-org / mlr3tuning

Hyperparameter optimization package of the mlr3 ecosystem

https://mlr3tuning.mlr-org.com/

GNU Lesser General Public License v3.0

53 stars 5 forks source link

Closed MichaelChirico closed 2 years ago

MichaelChirico commented 2 years ago

doing so is an error (at least on R 3.6.3)

I'm surprised this isn't triggered already, given the library(checkmate) in testthat.R 🤔

mllg commented 2 years ago

Not sure why we picked attachNamespace() here, but I guess we can keep things simple here and just call library("checkmate").

mllg commented 2 years ago

Thanks, fixed via library().