Skip to content

docs(readme): install follow-ups — shell reopen, audience note, Docker pointers - #215

Merged
vntrevx merged 1 commit into
mainfrom
docs/readme-install-notes
Aug 16, 2026
Merged

docs(readme): install follow-ups — shell reopen, audience note, Docker pointers#215
vntrevx merged 1 commit into
mainfrom
docs/readme-install-notes

Conversation

@vntrevx

@vntrevx vntrevx commented Aug 16, 2026

Copy link
Copy Markdown
Owner

Reader feedback on the install flow.

  • Install section: note that installation targets advanced users comfortable with the shell and, for data downloads or Freqtrade confirmation, with Docker and basic Ubuntu/Linux commands.
  • Install section: tell users to close and reopen the terminal (or open a new shell) after installation so nfi-bte is picked up on PATH before verifying.
  • Requirements: Docker bullet links the official Docker Engine install docs (per-distro commands: Ubuntu, Debian, Fedora, etc.) and states the installer does not manage Docker.

No installer change: install.sh is POSIX sh shared by Linux and macOS (Docker on macOS is Docker Desktop), Docker install needs sudo while the installer is user-level (uv tool), and Docker is only needed for data downloads / official Freqtrade confirmation, not the engine itself.

Documentation-only change; risk-tiered fast text/JSON lane per docs/ci-policy.md.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@vntrevx
vntrevx merged commit f24d055 into main Aug 16, 2026
7 checks passed
@vntrevx
vntrevx deleted the docs/readme-install-notes branch August 16, 2026 18:43
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