Any particular reason to move this into the class itself so that repo_file is not longer a class parameter? I'm fine with it either way -- just curious.