Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Navadeep0007
Copy link

@Navadeep0007 Navadeep0007 commented Oct 30, 2025

πŸ“ Description

This PR enhances the installation documentation by adding Ubuntu-specific Docker prerequisites. Many Ubuntu users struggle with Docker installation before they can start using Appwrite, so this addition provides clear, step-by-step instructions.

🎯 Type of Change

  • Documentation improvement
  • Bug fix
  • New feature
  • Breaking change

✨ Changes Made

  • Added comprehensive Ubuntu Docker installation guide
  • Included commands for adding user to docker group (to avoid sudo)
  • Added Docker verification step
  • Included important note about group changes requiring logout

πŸ§ͺ Testing

  • Tested all commands on Ubuntu 22.04 LTS
  • Verified Docker installs correctly using these steps
  • Confirmed all commands follow official Docker documentation
  • Checked markdown formatting and syntax

πŸ“Έ Screenshots

N/A - Documentation only

βœ… Checklist

  • My changes follow the project's style guidelines
  • I have performed a self-review of my changes
  • I have tested the documentation steps
  • My changes generate no new warnings
  • The documentation is clear and beginner-friendly

πŸ”— Related Issues

This helps Ubuntu users who need Docker prerequisites before installing Appwrite.

πŸ“š Additional Context

These instructions are based on Docker's official Ubuntu installation guide and have been tested on Ubuntu 22.04 LTS. They should work on Ubuntu 20.04+ versions. This will save Ubuntu users time by providing all necessary prerequisites in one place, rather than requiring them to search for Docker installation guides separately.

πŸ’‘ For Reviewers

This is a documentation-only change that adds significant value for the Ubuntu user community. The commands follow Docker's official installation procedures. No code or functionality changes are involved - purely documentation enhancement to improve the onboarding experience for Ubuntu users.


Thank you for reviewing! πŸ™

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 30, 2025

πŸ“ Walkthrough

Walkthrough

This pull request updates README.md to add an "Ubuntu Prerequisites" subsection under Self-Hosting. The new section provides step-by-step Docker installation instructions for Ubuntu (apt update, install dependencies, add Docker GPG key and repository, install Docker, enable/start the service, add the user to the docker group, refresh group membership, and verify installation). No code or exported/public entities are modified; the change is documentation-only.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Rationale: Documentation-only, homogeneous instructional content with no logic changes or code to verify.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
βœ… Passed checks (2 passed)
Check name Status Explanation
Title Check βœ… Passed The pull request title "docs: add Ubuntu prerequisites for Docker installation" directly and clearly summarizes the main change in the changeset. The raw summary confirms that the PR adds a new "Ubuntu Prerequisites" subsection with a full Ubuntu Docker installation guide to the README documentation. The title is concise, specific, and avoids vague terminology, accurately reflecting the primary objective of the PR.
Description Check βœ… Passed The pull request description is directly related to the changeset and provides clear, specific information about the changes. The author describes adding Ubuntu-specific Docker prerequisites to the installation documentation, which aligns with the raw summary showing a new "Ubuntu Prerequisites" subsection has been added to README.md with step-by-step Docker installation instructions. The description is not vague or genericβ€”it explicitly details the Ubuntu Docker installation guide, docker group configuration, and verification steps that were implemented. The description also provides appropriate context about testing on Ubuntu 22.04 LTS and following official Docker procedures.
✨ Finishing touches
  • πŸ“ Generate docstrings
πŸ§ͺ Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❀️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Oct 30, 2025

Security Scan Results for PR

Docker Image Scan Results

Package Version Vulnerability Severity
binutils 2.44-r2 CVE-2025-5244 HIGH
binutils 2.44-r2 CVE-2025-5245 HIGH
libxml2 2.13.8-r0 CVE-2025-49794 CRITICAL
libxml2 2.13.8-r0 CVE-2025-49796 CRITICAL
libxml2 2.13.8-r0 CVE-2025-49795 HIGH
libxml2 2.13.8-r0 CVE-2025-6021 HIGH
pcre2 10.43-r1 CVE-2025-58050 CRITICAL
golang.org/x/crypto v0.31.0 CVE-2025-22869 HIGH
golang.org/x/oauth2 v0.24.0 CVE-2025-22868 HIGH
stdlib 1.22.10 CVE-2025-47907 HIGH
stdlib 1.22.10 CVE-2025-47912 HIGH
stdlib 1.22.10 CVE-2025-58183 HIGH
stdlib 1.22.10 CVE-2025-58185 HIGH
stdlib 1.22.10 CVE-2025-58186 HIGH
stdlib 1.22.10 CVE-2025-58187 HIGH
stdlib 1.22.10 CVE-2025-58188 HIGH
stdlib 1.22.10 CVE-2025-58189 HIGH
stdlib 1.22.10 CVE-2025-61723 HIGH
stdlib 1.22.10 CVE-2025-61724 HIGH
stdlib 1.22.10 CVE-2025-61725 HIGH

Source Code Scan Results

πŸŽ‰ No vulnerabilities found!

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

πŸ“œ Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between a869ae3 and bdf34e7.

πŸ“’ Files selected for processing (1)
  • README.md (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
πŸ“š Learning: 2025-07-08T01:20:14.364Z
Learnt from: stnguyen90
PR: appwrite/appwrite#10119
File: app/controllers/api/account.php:1226-1232
Timestamp: 2025-07-08T01:20:14.364Z
Learning: In Appwrite, `_APP_DOMAIN` is a required environment variable that must always be set for the system to function properly.

Applied to files:

  • README.md
πŸͺ› markdownlint-cli2 (0.18.1)
README.md

78-78: Heading levels should only increment by one level at a time
Expected: h3; Actual: h4

(MD001, heading-increment)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: scan
πŸ”‡ Additional comments (1)
README.md (1)

78-111: Content quality is good; commands follow Docker's official guide.

The step-by-step instructions are well-organized with helpful inline comments, and the commands accurately reflect Docker's official Ubuntu installation process. The important note about logout/login requirement at lines 110–111 is appropriately placed and beneficial for users.

@Navadeep0007
Copy link
Author

Hi team ,
This PR adds Ubuntu-specific Docker prerequisites to the installation documentation. It’s a documentation-only change aimed at improving the onboarding experience for Ubuntu users by providing clear, step-by-step Docker setup instructions.

All commands have been tested on Ubuntu 22.04 LTS and verified against Docker’s official documentation. There are no code or functional changes β€” just improved clarity for new users.

Could someone with write access please review and approve this PR when possible?

Thank you for your time and support!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant