Bioconductor / ExperimentHub

Client to access ExperimentHub resources
https://bioconductor.org/packages/ExperimentHub
9 stars 13 forks source link

Misleading error message upon lack of connectivity #12

Closed LTLA closed 4 years ago

LTLA commented 4 years ago

Currently, we have:

https://github.com/Bioconductor/ExperimentHub/blob/9cb8394a834642a5d1f1192226d4b2ac60f946c4/R/ExperimentHub-class.R#L28

This wording suggests that the function was already using localHub=TRUE. I would suggest:

"No internet connection detected, switching to 'localHub=TRUE' instead"
lshep commented 4 years ago

Fixed in https://github.com/Bioconductor/ExperimentHub/commit/c4c9fc7cc12ed895cb572d8707dab4a1fefad286