remove extra forward slash unix

Bioconductor / AnnotationHub

Client for the Bioconductor AnnotationHub web resource

15 stars 13 forks source link

remove extra forward slash unix #3

Closed LiNk-NY closed 6 years ago

LiNk-NY commented 6 years ago

Minor change

mtmorgan commented 6 years ago

See the log associated with the line via git blame --

commit 3c2349d1352abcac9f54cf4d63d29b209f6f8f18
Author: Valerie Obenchain <vobencha@fhcrc.org>
Date:   Tue Aug 30 19:08:57 2016 +0000

    modify cache path to avoid creating a '~' directory on Mac

so any patch would need to be OS (not just unix) specific.

LiNk-NY commented 6 years ago

Hi Martin, @mtmorgan

It's a minor cosmetic issue but without being able to test the difference in the calls on Mac, I opted to remove the second foward slash / in the subsequent call.

The code here is also duplicated in ExperimentHub and can be changed there as well.

Regards, Marcel

mtmorgan commented 6 years ago

I'll close this without acting on it; without comprehensive testing the possibility of regression exceeds the cosmetic benefit.