Skip to content

Update README.md - #1

Merged
Nael-Nathanael merged 2 commits into
mainfrom
rndm-jpg-patch-1
Jul 14, 2026
Merged

Update README.md#1
Nael-Nathanael merged 2 commits into
mainfrom
rndm-jpg-patch-1

Conversation

@rndm-jpg

Copy link
Copy Markdown
Contributor

No description provided.

Prerequisites now match docs/running.md: Docker + Docker Compose
provide Postgres, Redis, and minio via compose, so they are no longer
listed as host installs. Pin Node.js 20+ and Python 3.11+, and add
minio to the architecture list.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@Nael-Nathanael Nael-Nathanael left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved. Reviewed the full README rewrite against the repo: all referenced files exist (ARCHITECTURE.md, docs/running.md, docs/roadmap.md, CONTRIBUTING.md, SECURITY.md, logo path), and every Make target in the quickstart is real.

One factual issue was found and fixed directly on this branch in 3a423e8: the prerequisites listed PostgreSQL and Redis as host installs, but make db-up provides Postgres, Redis, and minio via Docker Compose. Prerequisites now match docs/running.md (Docker + Docker Compose, Node.js 20+, Python 3.11+), and minio was added to the architecture list.

Good to merge.

@Nael-Nathanael
Nael-Nathanael merged commit b18c419 into main Jul 14, 2026
2 checks passed
@Nael-Nathanael
Nael-Nathanael deleted the rndm-jpg-patch-1 branch July 14, 2026 16:31
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.

2 participants