Skip to content

Add UMASK environment variable to README#94

Merged
nitrobass24 merged 1 commit intodevelopfrom
docs/add-umask-to-readme
Feb 28, 2026
Merged

Add UMASK environment variable to README#94
nitrobass24 merged 1 commit intodevelopfrom
docs/add-umask-to-readme

Conversation

@nitrobass24
Copy link
Owner

@nitrobass24 nitrobass24 commented Feb 28, 2026

Summary

  • Add UMASK to the docker-compose example as a commented-out option
  • Add UMASK comment to the docker run example
  • Add UMASK row to the environment variables table

The UMASK feature was added in 6fade95 but the README was not updated.

Test plan

  • Verify README renders correctly on GitHub

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Added documentation for the optional UMASK environment variable, which controls file permissions with Docker Compose and docker run examples provided.

Document the UMASK option in the docker-compose example,
docker run example, and environment variables table.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@nitrobass24 nitrobass24 merged commit a976b9e into develop Feb 28, 2026
4 checks passed
@nitrobass24 nitrobass24 deleted the docs/add-umask-to-readme branch February 28, 2026 21:24
@coderabbitai
Copy link

coderabbitai bot commented Feb 28, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b5be932 and 2126a12.

📒 Files selected for processing (1)
  • README.md

📝 Walkthrough

Walkthrough

Documentation updated to introduce the optional UMASK environment variable for controlling file permissions in Docker environments. Added examples showing UMASK usage in Docker Compose and docker run configurations, along with corresponding permission outputs.

Changes

Cohort / File(s) Summary
Documentation
README.md
Added UMASK environment variable documentation with Docker usage examples and permission mapping (002→775/664, 000→777/666).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Permissions now dance at your command,
With UMASK we grant or withstand,
775 or 777, just take your pick,
Documentation makes configs click!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch docs/add-umask-to-readme

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

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