-
Notifications
You must be signed in to change notification settings - Fork 147
FIX : docs: add missing PORT variable to env example in readme #677
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
Conversation
WalkthroughThe README.md file was updated to include the Changes
Poem
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
README.md (1)
278-289: Add a brief inline comment and keep port variables clustered for readability
A one-liner comment (like the other blocks often use) clarifies what this port controls, and grouping it with the other exposed service ports improves scan-ability.ANALYTICS_SERVER_PORT=9696 SYNC_SERVER_PORT=9697 -PORT=3333 +# Frontend / web-server port +PORT=3333
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md(1 hunks)
🔇 Additional comments (1)
README.md (1)
288-288: Addition aligns env documentation with actual.env.example– good catch
ThePORT=3333line removes the discrepancy and prevents “missing PORT” errors when runningyarn dev.
|
Strange because it should just take the port by default if not supplied. |
This PR makes a minor update to the README to include the missing
PORTenvironment variable required for running the frontend locally usingyarn dev.I used the steps in the README to setup middleware locally and faced the following error.

.env.example file has it right. Its just missing in the README.
Let me know you if you have any suggestions.
Thanks,
Ritik
Summary by CodeRabbit
PORT=3333environment variable in the example configuration for running middleware locally.