search

DomiStyle / docker-tor-browser

Tor Browser (in a browser)
https://hub.docker.com/r/domistyle/tor-browser
MIT License
80 stars 23 forks source link

Shouldn't need to change the download URLs #15

Closed davidnewcomb closed 3 years ago

davidnewcomb commented 3 years ago

I got here first this time and was sick of changing the version number in 5 different places! So I fixed it. The next person only has to change it in 1 place :) We also get a prettier title out of it too, so there is a bonus.

As it's a new feature, I probably ought to have done it on a branch with a merge request and we could have had a chat about it but as it was only a line or two, I just did it on master, hope that's ok?

DomiStyle commented 3 years ago

I don't see any problems.

That's how it was supposed to work in the first place, I was just in a hurry. :)

ghost commented 3 years ago

Apologies, I've been in the hospital for the past month. The change @davidnewcomb made LGTM as well.