Skip to content

fix: Broken install instructions#3937

Open
AlexDev404 wants to merge 1 commit intoDokploy:canaryfrom
AlexDev404:fix/broken-install-instructions
Open

fix: Broken install instructions#3937
AlexDev404 wants to merge 1 commit intoDokploy:canaryfrom
AlexDev404:fix/broken-install-instructions

Conversation

@AlexDev404
Copy link
Contributor

@AlexDev404 AlexDev404 commented Mar 8, 2026

What is this PR about?

This fixes the issue where the installation instructions are instructing to have the user run the install script under the wrong shell.

Checklist

Before submitting this PR, please make sure that:

  • You created a dedicated branch based on the canary branch.
  • You have read the suggestions in the CONTRIBUTING.md file https://github.com/Dokploy/dokploy/blob/canary/CONTRIBUTING.md#pull-request
  • You have tested this PR in your local instance. If you have not tested it yet, please do so before submitting. This helps avoid wasting maintainers' time reviewing code that has not been verified by you.

Greptile Summary

This PR updates the installation command in README.md to explicitly use bash instead of sh when piping the install script. This is a valid documentation fix: on many Linux distributions (e.g. Ubuntu, Debian), sh resolves to dash, which lacks bash-specific features that the install script likely relies on, causing the install to fail or behave unexpectedly. The markdown code block was already annotated as bash, so the command now matches the declared shell, eliminating a source of installation failures.

Confidence Score: 5/5

  • This PR is safe to merge — it is a single-line documentation fix with no code changes, configuration updates, or logic modifications.
  • The change is minimal and correct: replacing sh with bash in the install command aligns the actual shell invoked with the code block language hint and prevents installation failures on systems where sh is dash. No source code, logic, or configuration is affected. The fix directly addresses a known issue in the installation instructions.
  • No files require special attention.

Last reviewed commit: 2da45d3

This fixes the issue where the installation instructions are instructing to have the user run the install script under the wrong shell.
@AlexDev404 AlexDev404 requested a review from Siumauricio as a code owner March 8, 2026 08:02
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. documentation Improvements or additions to documentation labels Mar 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant