[PR #198] [MERGED] Update docker docs to remove sudo #200

Closed
opened 2026-03-02 02:51:23 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/amidaware/trmm-docs/pull/198
Author: @grahamIT
Created: 8/29/2023
Status: Merged
Merged: 8/29/2023
Merged by: @wh1te909

Base: mainHead: main


📝 Commits (4)

📊 Changes

2 files changed (+22 additions, -6 deletions)

View changed files

📝 docs/install_docker.md (+9 -1)
📝 docs/update_docker.md (+13 -5)

📄 Description

I noticed that the docker install and update docs suggested using docker as root. This is a security risk and is unnecessarily complicated. The better alternative is to add your user into the docker group. This allows for easy access to docker commands without having to deal with sudo or passwords.

I also added a bit of clarification for installing docker.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/amidaware/trmm-docs/pull/198 **Author:** [@grahamIT](https://github.com/grahamIT) **Created:** 8/29/2023 **Status:** ✅ Merged **Merged:** 8/29/2023 **Merged by:** [@wh1te909](https://github.com/wh1te909) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (4) - [`4b4602e`](https://github.com/amidaware/trmm-docs/commit/4b4602e46a3ff4ab6c94abf84e84b7f02ef9e934) Use docker-compose directly - [`59a40e5`](https://github.com/amidaware/trmm-docs/commit/59a40e54d0d012128bff964fa37c27408d41af83) remove sudo - [`51cd9a1`](https://github.com/amidaware/trmm-docs/commit/51cd9a164fa958042257402eaf0076f796c01b1f) remove sudo - [`79060cd`](https://github.com/amidaware/trmm-docs/commit/79060cdcaadfb999ac6838e20eb693515c95d6f7) add clarification to docker install section ### 📊 Changes **2 files changed** (+22 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `docs/install_docker.md` (+9 -1) 📝 `docs/update_docker.md` (+13 -5) </details> ### 📄 Description I noticed that the docker install and update docs suggested using docker as root. This is a security risk and is unnecessarily complicated. The better alternative is to add your user into the docker group. This allows for easy access to docker commands without having to deal with sudo or passwords. I also added a bit of clarification for installing docker. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-02 02:51:23 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/trmm-docs#200
No description provided.