Skip to content

Fix docker compose in readme#4091

Merged
ildyria merged 1 commit intomasterfrom
fix-readme
Feb 15, 2026
Merged

Fix docker compose in readme#4091
ildyria merged 1 commit intomasterfrom
fix-readme

Conversation

@ildyria
Copy link
Member

@ildyria ildyria commented Feb 15, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

Release Notes

  • Documentation
    • Updated Docker Compose volume mappings for improved storage organization of logs and temporary files.
    • Added setup instructions for APP_KEY generation using OpenSSL.

@ildyria ildyria requested a review from a team as a code owner February 15, 2026 18:35
@coderabbitai
Copy link

coderabbitai bot commented Feb 15, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

The README.md documentation is updated to modify docker-compose volume mappings from generic storage paths to specific log and temporary directory paths. Comments are added explaining APP_KEY generation using openssl for configuration setup.

Changes

Cohort / File(s) Summary
Documentation
README.md		 Updated docker-compose volume mappings from ./lychee/storage:/app/storage and .env:/app/.env:ro to ./lychee/logs:/app/storage/logs and ./lychee/tmp:/app/storage/tmp. Added inline comments with APP_KEY generation instructions using openssl and configuration guidance.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A README so fine, with paths now refined,
Logs and temps neatly aligned,
OpenSSL wisdom in comments divine,
Docker volumes in order—a blueprint sublime! ✨

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into master

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

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.

coderabbitai[bot]
coderabbitai bot requested changes Feb 15, 2026
View reviewed changes
Copy link

@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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
README.md (1)

145-145: 🛠️ Refactor suggestion | 🟠 Major

Add "Last updated" footer per coding guidelines.

The coding guidelines require that documentation files include a horizontal rule followed by a "Last updated: [date of the update]" line at the bottom. This footer is currently missing.

As per coding guidelines: "At the bottom of documentation files, add an hr line followed by 'Last updated: [date of the update]'"

📝 Proposed addition at end of file

Add after line 145:

 [discord-shield]: https://img.shields.io/discord/1046911561366765598?logo=discord
+
+---
+*Last updated: 2026-02-15*

@ildyria ildyria merged commit 84d015f into master Feb 15, 2026
4 checks passed
@ildyria ildyria deleted the fix-readme branch February 15, 2026 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ildyria