Skip to content

fix: formatting#205

Merged
sriramveeraghanta merged 1 commit intomasterfrom
formatting-issues
Feb 5, 2026
Merged

fix: formatting#205
sriramveeraghanta merged 1 commit intomasterfrom
formatting-issues

Conversation

@sriramveeraghanta
Copy link
Copy Markdown
Member

@sriramveeraghanta sriramveeraghanta commented Feb 5, 2026

Summary by CodeRabbit

Release Notes

  • Documentation
    • Fixed typos in self-hosting installation guides (Coolify, Docker Swarm, Portainer)
    • Restructured Docker Swarm deployment guide with improved formatting, explicit code blocks, and clearer step sequencing for easier setup and reference

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 5, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
developer-docs Ready Ready Preview, Comment Feb 5, 2026 3:08pm

Request Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

Documentation updates across three self-hosting guides: typo corrections ("depoyment" → "deployment") in Coolify and Portainer guides, and restructuring of the Docker Swarm guide with reformatted steps, explicit code blocks, and numbered step sequencing for improved clarity.

Changes

Cohort / File(s) Summary
Typo Fixes
docs/self-hosting/methods/coolify.md, docs/self-hosting/methods/portainer.md
Corrected misspelling "depoyment" to "deployment" in procedure sections.
Docker Swarm Guide Restructuring
docs/self-hosting/methods/docker-swarm.md
Reformatted deployment guide with explicit code blocks for curl commands, expanded 2-step process into 6-step sequence, added dedicated "Load environment variables" step, and improved formatting while preserving original behavior.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A typo's been caught, how delightful indeed!
Deployment, not "depoyment"—just what we all need.
Docker Swarm's steps now shine bright and clear,
With code blocks and structure, the path is sincere.
Documentation perfected, hopping with cheer! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The PR title 'fix: formatting' is vague and does not clearly describe the main changes, which include multiple typo fixes and substantial reformatting across three documentation files. Use a more descriptive title that captures the primary changes, such as 'fix: correct typos and improve formatting in self-hosting documentation' or 'docs: fix typos and restructure Docker Swarm deployment guide'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch formatting-issues

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.

@sriramveeraghanta sriramveeraghanta merged commit 49a7fc8 into master Feb 5, 2026
2 of 3 checks passed
@sriramveeraghanta sriramveeraghanta deleted the formatting-issues branch February 5, 2026 15:09
Copy link
Copy Markdown

@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

🤖 Fix all issues with AI agents
In `@docs/self-hosting/methods/docker-swarm.md`:
- Around line 27-58: The doc downloads the env file as plane.env but later
references variables.env (e.g., the sed command and the load step using source
<path-to variables.env>), causing a filename mismatch; make the filenames
consistent by either changing the download command to save as variables.env or
updating all subsequent references (the sed line updating MACHINE_SIGNATURE and
the load step using set -o allexport; source ...) to use plane.env so every
mention (variables.env / plane.env) matches across the document.

Comment on lines +27 to +58
- `variables.env` – Stores environment variables for your deployment.

- `DOMAIN_NAME` – (required) Your application's domain name.
- `SITE_ADDRESS` – (required) The full domain name (FQDN) of your instance.
- `MACHINE_SIGNATURE` – (required) A unique identifier for your machine. You can generate this by running below code in terminal:
```sh
sed -i 's/MACHINE_SIGNATURE=.*/MACHINE_SIGNATURE='$(openssl rand -hex 16)'/' plane.env
```
- `CERT_EMAIL` – (optional) Email address for SSL certificate generation (only needed if you're setting up HTTPS).
```bash
curl -fsSL https://prime.plane.so/releases/<plane-version>/variables.env -o plane.env
```

::: warning
The `<plane-version>` value should be v1.8.3 or higher.
:::

2. **Configure environment variables**
Before deploying, edit the `variables.env` file in your preferred text editor and update the following values:
- `DOMAIN_NAME` – (required) Your application's domain name.
- `SITE_ADDRESS` – (required) The full domain name (FQDN) of your instance.
- `MACHINE_SIGNATURE` – (required) A unique identifier for your machine. You can generate this by running below code in terminal:
```sh
sed -i 's/MACHINE_SIGNATURE=.*/MACHINE_SIGNATURE='$(openssl rand -hex 16)'/' plane.env
```
- `CERT_EMAIL` – (optional) Email address for SSL certificate generation (only needed if you're setting up HTTPS).

3. **Configure external DB, Redis, and RabbitMQ**
::: warning
When self-hosting Plane for production use, it is strongly recommended to configure external database and storage. This ensures that your data remains secure and accessible even if the local machine crashes or encounters hardware issues. Relying solely on local storage for these components increases the risk of data loss and service disruption.
:::
::: warning
When self-hosting Plane for production use, it is strongly recommended to configure external database and storage. This ensures that your data remains secure and accessible even if the local machine crashes or encounters hardware issues. Relying solely on local storage for these components increases the risk of data loss and service disruption.
:::
- `DATABASE_URL` – Connection string for your external database.
- `REDIS_URL` – Connection string for your external Redis instance.
- `AMQP_URL` – Connection string for your external RabbitMQ server.

4. **Load the environment variables**

```bash
set -o allexport; source <path-to variables.env>; set +o allexport;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Fix filename inconsistency for env file.
You download variables.env as plane.env, but later steps refer to variables.env. This mismatch will confuse users and break the copy‑paste flow.

✅ Suggested doc fix
-   - `variables.env` – Stores environment variables for your deployment.
+   - `plane.env` – Stores environment variables for your deployment.

-   Before deploying, edit the `variables.env` file in your preferred text editor and update the following values:
+   Before deploying, edit the `plane.env` file in your preferred text editor and update the following values:

-   set -o allexport; source <path-to variables.env>; set +o allexport;
+   set -o allexport; source <path-to plane.env>; set +o allexport;
🤖 Prompt for AI Agents
In `@docs/self-hosting/methods/docker-swarm.md` around lines 27 - 58, The doc
downloads the env file as plane.env but later references variables.env (e.g.,
the sed command and the load step using source <path-to variables.env>), causing
a filename mismatch; make the filenames consistent by either changing the
download command to save as variables.env or updating all subsequent references
(the sed line updating MACHINE_SIGNATURE and the load step using set -o
allexport; source ...) to use plane.env so every mention (variables.env /
plane.env) matches across the document.

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.

2 participants