Skip to content

docs: Clarify the first FlatRun deployment - #221

Merged
nfebe merged 4 commits into
mainfrom
docs/task-first-positioning
Aug 24, 2026
Merged

docs: Clarify the first FlatRun deployment#221
nfebe merged 4 commits into
mainfrom
docs/task-first-positioning

Conversation

@nfebe

@nfebe nfebe commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@sourceant

sourceant Bot commented Aug 24, 2026

Copy link
Copy Markdown

Code Review Summary

Este PR mejora la documentación inicial de FlatRun, reorganizando el README principal y añadiendo traducciones completas para varios idiomas. La estructura es ahora más clara y orientada a la acción.

🚀 Key Improvements

  • Reorganización del README.md para destacar el flujo 'del servidor a la aplicación'.
  • Traducción completa a Español, Francés, Portugués y Chino Simplificado.
  • Consistencia en los pasos de instalación entre la UI y el CLI.

💡 Minor Suggestions

  • Alinear los comandos de instalación del CLI en las traducciones con las mejoras solicitadas para la versión en inglés.

🚨 Critical Issues

  • Inconsistencia en el shell (sh vs bash) y puertos de ejemplo en las instrucciones del CLI en los archivos de traducción.

@sourceant sourceant Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

Comment thread README.md
agent API:

```bash
curl -fsSL https://raw.githubusercontent.com/flatrun/cli/main/scripts/install.sh | sudo sh

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

For consistency with the agent installation command and to ensure the installer script executes in the intended environment, use bash instead of sh. Additionally, using a placeholder that reflects the default Agent API port (8090) makes the first-time setup clearer for users who have just followed the initial server installation steps.

Suggested change
curl -fsSL https://raw.githubusercontent.com/flatrun/cli/main/scripts/install.sh | sudo sh
curl -fsSL https://raw.githubusercontent.com/flatrun/cli/main/scripts/install.sh | sudo bash
flatrun profile add production --url http://<your-server>:8090 --token your-api-key-here

@sourceant sourceant Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review complete. No specific code suggestions were generated. See the overview comment for a summary.

@nfebe
nfebe merged commit aca6e91 into main Aug 24, 2026
1 check passed
@nfebe
nfebe deleted the docs/task-first-positioning branch August 24, 2026 23:05
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