Possible issue in detach.packages() when trying to detach a package which is depended on by a loaded package

[colinpmillar]

TAF/R/detach.packages.R

Line 30 in 0e7c09f

sapply(paste0("package:",pkgs), detach, character.only=TRUE)

[arni-magnusson]

The 'detach' argument in source.taf(), defaulting to FALSE is something I haven't used or tested much. It looks we added that feature back in April 2020, so it probably seemed like a good idea at the time :)

Happy to look at a reproducible example and/or talk about possible improvements. Currently, I don't have test cases where detach=TRUE can be helpful or harmful.

My guess is that few people run source.taf() with detach=TRUE, but I notice that taf=TRUE enables detach=TRUE. Does that mean that the TAF server runs all analyses with detach=TRUE?