benmarwick / rrtools

rrtools: Tools for Writing Reproducible Research in R
Other
671 stars 85 forks source link

username for dockerhub from env. variables #17

Closed MartinHinz closed 7 years ago

MartinHinz commented 7 years ago

With this the username for hub.docker.com is taken from the travis env variables instead of taken the github user name.

Referring to our discussion in https://github.com/benmarwick/rrtools/issues/12, this actually seems to be more elegant than entering it locally since:

benmarwick commented 7 years ago

Thanks, yes, very elegant! Excellent idea! The could be a problem in the case where a user wants to use docker without travis (for example make a docker container locally on their computer), but let's not worry about that for now because that has other issues to consider also.

MartinHinz commented 7 years ago

The only file affected is the travis.yml, while the dockerfile itself does not contain any information about the user. So there should be no problems in respect to this.? What about closing #12 then?

benmarwick commented 7 years ago

Thanks, closes #12