-
Notifications
You must be signed in to change notification settings - Fork 107
Add Linux Distribution Feature Comparison Table (Fixes #40) #47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -17,6 +17,7 @@ | |||||
|
|
||||||
| ### One-Click Installation Script | ||||||
| This script streamlines the installation process, providing a seamless experience for various operating systems. Whether you prefer a graphical interface or the command line, it's designed for simplicity and efficiency.Supported OS: | ||||||
| > Below are the Linux distributions you can install using LinuxDroid. | ||||||
| # Copy One-Click Installation Script | ||||||
| ``` | ||||||
| curl -sSL https://raw.githubusercontent.com/AryanVBW/LinuxDroid/refs/heads/main/1click.install.sh | bash | ||||||
|
|
@@ -35,6 +36,26 @@ This script copies the installation command for LinuxDroid to your clipboard, ma | |||||
| - NethunterRootLess (Hacking specialized Linux)<img src="https://github.com/AryanVBW/Myimage/releases/download/m3/kali-nethunter-dragon-tm.png" height="12"> | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||||||
| - BlackBox(coming🔜)<img src="https://raw.githubusercontent.com/AryanVBW/LinuxDroid/main/LinuxDroid/Icon/bosckbocBg.png" height="12"> | ||||||
| - CentOS(coming 🔜)<img src="https://raw.githubusercontent.com/AryanVBW/LinuxDroid/main/LinuxDroid/Icon/centosbg.png" height="12"> | ||||||
|
|
||||||
| --- | ||||||
| ### Linux Distribution Comparison | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win 🧩 Analysis chain🏁 Script executed: #!/bin/bash
set -euo pipefail
echo "== README lines around headings =="
nl -ba README.md | sed -n '1,80p;160,190p'
echo
echo "== markdown heading-looking lines =="
python3 - <<'PY'
from pathlib import Path
import re
p=Path("README.md")
for i,line in enumerate(p.read_text(errors="replace").splitlines(),1):
if re.match(r'^#{1,6}\s*|^#\S', line):
print(f"{i}: {line}")
PY
echo
echo "== linting tools availability =="
(command -v mdl || true) && mdl -s . || true
(command -v markdownlint || true) && markdownlint README.md || true
(command -v node || true) && node -v || trueRepository: AryanVBW/LinuxDroid Length of output: 232 🏁 Script executed: #!/bin/bash
set -u
echo "== locate README =="
find . -maxdepth 3 -type f -name README.md -print
echo
echo "== lines around headings =="
python3 - <<'PY'
from pathlib import Path
for path in Path('.').rglob('README.md'):
print(f"\n--- {path} ---")
lines = path.read_text(errors="replace").splitlines()
for i in range(0, min(len(lines),80)):
print(f"{i+1}: {lines[i]}")
for low in [160,170]:
if low <= len(lines):
print(f"\n--- lines {low}-{min(len(lines),190)} ---")
for j in range(low-1, min(190, len(lines))):
print(f"{j+1}: {lines[j]}")
print("\n--- heading-looking lines ---")
import re
for i,line in enumerate(lines,1):
if re.match(r'^#{1,6}(?:[^\h]|\h.*$)', line):
print(f"{i}: {line}")
PY
echo
echo "== linting tools availability =="
for cmd in mdls mdl markdownlint node; do
if command -v "$cmd" >/dev/null 2>&1; then
echo "$cmd: $(command -v "$cmd")"
if [ "$cmd" = node ]; then node -v; fi
else
echo "$cmd: not found"
fi
doneRepository: AryanVBW/LinuxDroid Length of output: 10318 Normalize the README headings.
🧰 Tools🪛 markdownlint-cli2 (0.23.2)[warning] 41-41: Heading levels should only increment by one level at a time (MD001, heading-increment) 🤖 Prompt for AI AgentsSource: Linters/SAST tools |
||||||
|
|
||||||
| Quick look at RAM, storage, and desktop environments for each distro. | ||||||
|
|
||||||
| | Distribution | RAM | Storage | Desktop Environments | Best For | | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P3: The PR description states the comparison table includes a 'difficulty level' column, but the added table header only has Distribution, RAM, Storage, Desktop Environments, and Best For — there's no per-distribution Difficulty column. Add the missing column or update the PR description to match. Prompt for AI agents |
||||||
| |--------------|-----|---------|----------------------|----------| | ||||||
| | <img src="https://raw.githubusercontent.com/AryanVBW/LinuxDroid/main/LinuxDroid/Icon/linux-mint.svg" height="16"> **Linux Mint** | 2 GB | 15 GB | Cinnamon, MATE, XFCE | Beginners, everyday use | | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P2: The new comparison table presents Linux Mint and Pop!_OS as LinuxDroid install targets and the tip recommends Mint, but neither is in the actual supported-installation list in this same README (Parrot, Kali, Ubuntu, Arch, Debian, Manjaro, Void, Alpine, Fedora, NetHunterRootLess; BlackBox/CentOS coming soon). Users following this section may try to install Mint or Pop!_OS and fail. Either restrict the table to the distros LinuxDroid actually installs (adding Parrot, Manjaro, Void, Alpine) or clearly mark Mint/Pop!_OS as reference-only and not installable. Prompt for AI agentsThere was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P3: These table icons lack Prompt for AI agents
Suggested change
|
||||||
| | <img src="https://raw.githubusercontent.com/AryanVBW/LinuxDroid/main/LinuxDroid/Icon/ubuntu-logo.png" height="16"> **Ubuntu** | 2 GB | 25 GB | GNOME, KDE, XFCE | General use, productivity | | ||||||
| | <img src="https://raw.githubusercontent.com/AryanVBW/LinuxDroid/main/LinuxDroid/Icon/Fedorabg.png" height="16"> **Fedora** | 2 GB | 20 GB | GNOME, KDE, XFCE | Developers, newer packages | | ||||||
| | <img src="https://raw.githubusercontent.com/AryanVBW/LinuxDroid/main/LinuxDroid/Icon/debiBg.png" height="16"> **Debian** | 1 GB | 10 GB | GNOME, KDE, XFCE, LXDE | Servers, stability | | ||||||
| | <img src="https://raw.githubusercontent.com/AryanVBW/LinuxDroid/main/LinuxDroid/Icon/linux.png" height="16"> **Pop!_OS** | 4 GB | 20 GB | GNOME | Gaming, workstations | | ||||||
| | <img src="https://raw.githubusercontent.com/AryanVBW/LinuxDroid/main/LinuxDroid/Icon/Kali.png" height="16"> **Kali Linux** | 2 GB | 4 GB | XFCE, KDE | Pentesting, cybersecurity | | ||||||
| | <img src="https://raw.githubusercontent.com/AryanVBW/LinuxDroid/main/LinuxDroid/Icon/archBg.png" height="16"> **Arch Linux** | 1 GB | 5 GB | Any (manual setup) | Advanced users, custom builds | | ||||||
|
Comment on lines
+47
to
+53
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win Add alt text to the table icons. The images at Lines 47-53 have no Proposed fix-| <img src="..." height="16"> **Linux Mint** |
+| <img src="..." height="16" alt=""> **Linux Mint** |🧰 Tools🪛 markdownlint-cli2 (0.23.2)[warning] 47-47: Images should have alternate text (alt text) (MD045, no-alt-text) [warning] 48-48: Images should have alternate text (alt text) (MD045, no-alt-text) [warning] 49-49: Images should have alternate text (alt text) (MD045, no-alt-text) [warning] 50-50: Images should have alternate text (alt text) (MD045, no-alt-text) [warning] 51-51: Images should have alternate text (alt text) (MD045, no-alt-text) [warning] 52-52: Images should have alternate text (alt text) (MD045, no-alt-text) [warning] 53-53: Images should have alternate text (alt text) (MD045, no-alt-text) 🤖 Prompt for AI AgentsSource: Linters/SAST tools |
||||||
|
|
||||||
|
Owner
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove emojis and use icons instead. |
||||||
| > Tip: If you're new to Linux, start with Mint or Ubuntu. | ||||||
|
Comment on lines
+45
to
+55
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win Add the per-distribution difficulty level. The PR objective requires a difficulty level. The table has no 🧰 Tools🪛 markdownlint-cli2 (0.23.2)[warning] 47-47: Images should have alternate text (alt text) (MD045, no-alt-text) [warning] 48-48: Images should have alternate text (alt text) (MD045, no-alt-text) [warning] 49-49: Images should have alternate text (alt text) (MD045, no-alt-text) [warning] 50-50: Images should have alternate text (alt text) (MD045, no-alt-text) [warning] 51-51: Images should have alternate text (alt text) (MD045, no-alt-text) [warning] 52-52: Images should have alternate text (alt text) (MD045, no-alt-text) [warning] 53-53: Images should have alternate text (alt text) (MD045, no-alt-text) 🤖 Prompt for AI Agents |
||||||
|
|
||||||
| --- | ||||||
|
|
||||||
| ### Supported Desktop Environments: | ||||||
|
|
||||||
| - LXDE | ||||||
|
|
@@ -153,7 +174,7 @@ Maximum flexibility | |||||
| Install Kali NetHunter on any stock, unrooted Android device without voiding the warranty | ||||||
| Android Device (Stock unmodified device, no root or custom recovery required) | ||||||
| #📸 Screenshots | ||||||
| ## Tap on Arrow👇 | ||||||
| ### Tap on Arrow👇 | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P3: This changes the heading level from Prompt for AI agents
Suggested change
|
||||||
| <p align="center"> | ||||||
| <a href="https://github.com/AryanVBW/LinuxDroid/blob/L2/LooksN.md"> | ||||||
| <img src="https://raw.githubusercontent.com/AryanVBW/LinuxDroid/main/LinuxDroid/Arrow/1024arow2.png" height="120" alt="LinuxDroid"> | ||||||
|
|
||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
Repository: AryanVBW/LinuxDroid
Length of output: 249
🏁 Script executed:
Repository: AryanVBW/LinuxDroid
Length of output: 12533
Align installable distributions with the distribution table.
Line 20 says everything below is installable, but lines 37-38 mark BlackBox and CentOS as coming soon. The Linux Distribution Comparison table includes Linux Mint and Pop!_OS, and line 55 recommends Mint, even though those distributions do not appear in the supported install list. Move the missing distributions into the supported OS list with clear statuses, or remove them from the supported/comparison sections if they are not installable.
🤖 Prompt for AI Agents