fix expansion ADDITIONALFLAGS

[EvilOlaf]

Summary by CodeRabbit

• Documentation

  • Updated environment variable configuration examples for clarity.

• Chores

  • Adjusted docker-compose environment variable formatting.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Configuration formatting adjustments to ADDITIONALFLAGS across documentation and deployment files. README example removes surrounding quotes, while docker-compose environment variable representation is updated for consistency in how empty values are specified.

Changes

Cohort / File(s)	Summary
ADDITIONALFLAGS formatting updates README.md, docker-compose.yml	Removes quotes from ADDITIONALFLAGS value in README example ("-nobattleye" → -nobattleye). Updates docker-compose environment variable formatting for the empty value representation to ensure consistent parsing across configurations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• Add ADDITIONALFLAGS environment variable #29: Addresses the same ADDITIONALFLAGS variable and its representation across configuration files and server startup behavior.

Poem

🐰 Quotes removed with care, the config's now bare,
No extra marks cluttering the air,
Empty values shine, formatted fine,
Consistency blooms across each line! 🌿

Pre-merge checks

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change: fixing the expansion/handling of the ADDITIONALFLAGS variable across README.md and docker-compose.yml files.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

---

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between dfeebea and 86bb296.

📒 Files selected for processing (2)

• README.md
• docker-compose.yml

🔇 Additional comments (2)

README.md (1)

70-70: LGTM! Correct removal of quotes from the documentation example.

Removing the quotes around -nobattleye prevents them from being literally included in the environment variable value, which could cause issues during command expansion. Since the flag contains no spaces or special characters, quotes are unnecessary.

docker-compose.yml (1)

10-10: LGTM! Consistent with README documentation changes.

The change from ADDITIONALFLAGS="" to ADDITIONALFLAGS= is functionally equivalent in docker-compose (both set an empty string), but provides consistency with the quote removal in the README example. This ensures the environment variable expansion behaves correctly when users override this value.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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