-
Notifications
You must be signed in to change notification settings - Fork 4.4k
docs: improve documentation clarity and fix inconsistencies #665
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?
docs: improve documentation clarity and fix inconsistencies #665
Conversation
- Fix BETTER_AUTH_URL inconsistency (README used BETTER_AUTH_URL, CONTRIBUTING.md and .env.example use NEXT_PUBLIC_BETTER_AUTH_URL) - Add optional environment variables section to both README and CONTRIBUTING.md - Document Freesound, Cloudflare R2, and Modal transcription setup - Clarify which variables are required vs optional for local development - Fix typo in apps/transcription/README.md (accoumt -> account) These changes help new contributors understand the full setup process and avoid confusion about environment variables.
👷 Deploy request for appcut pending review.Visit the deploys page to approve it
|
|
@claudeaceae is attempting to deploy a commit to the OpenCut OSS Team on Vercel. A member of the Team first needs to authorize it. |
WalkthroughDocumentation files updated to clarify environment variables. Root README changes BETTER_AUTH_URL to NEXT_PUBLIC_BETTER_AUTH_URL and adds optional variables section for advanced features. Contributing guide receives new environment variable sections. Transcription README typo corrected. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes
Possibly related PRs
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (3)
🧰 Additional context used🪛 Gitleaks (8.30.0).github/CONTRIBUTING.md[high] 112-112: Detected a Generic API Key, potentially exposing access to various services and sensitive operations. (generic-api-key) README.md[high] 118-118: Detected a Generic API Key, potentially exposing access to various services and sensitive operations. (generic-api-key) 🔇 Additional comments (6)
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. Comment |
Summary
This PR improves the documentation by fixing inconsistencies and adding clarity around environment variable setup.
Changes Made
Fixed environment variable inconsistency
BETTER_AUTH_URLwhile CONTRIBUTING.md and.env.exampleuseNEXT_PUBLIC_BETTER_AUTH_URLAdded optional environment variables section
Fixed typo
Why These Changes?
These changes help new contributors:
Addresses
This partially addresses issue #10 by improving the getting started documentation.
Note: All changes are documentation-only. No code changes included.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.