docker-tor-browser icon indicating copy to clipboard operation
docker-tor-browser copied to clipboard

Readme Updated, Docker Compose Improved

Open GitXpresso opened this issue 1 year ago • 4 comments

I mostly updated the mark down the docker compose file got updated too. Looking back at my commits, I didn't check twice before I describe what I changed in the files.

GitXpresso avatar Nov 09 '24 04:11 GitXpresso

Pull request, readme.md got updated

GitXpresso avatar Nov 09 '24 05:11 GitXpresso

  1. There is no need to include a copy of docker-compose.yml in the README file. It's just a duplication that would need to be updated. Please remove that section.
  2. "To verify that you docker put the following command below" - ??
  3. Remove "Github user who updated this markdown:" heading and section - no personal accreditations.
  4. Adjust table of contents accordingly.
  5. Change restart: unless-stopped back to restart: always - no data must be left on the filesystem when you are not using it. The current version of Firefox allows you to update the browser without killing the container. If you would like a permanant update, then you must rebuild it. The rebuild will pick up the latest OS updates as well.
  6. Revert "Provides direct access to the NoVNC server" back to "Provides direct access to the VNC server" - it is a VNC server. NoVNC is a client.

davidnewcomb avatar Nov 11 '24 09:11 davidnewcomb

Everything's done just let me know if I need to do some tweaks/changes to the readme and compose.yml

GitXpresso avatar Dec 26 '24 05:12 GitXpresso

Sorry, haven't focus on this, I currently working on a huge project, LinuxApps NoVNC

GitXpresso avatar Jan 20 '25 14:01 GitXpresso