Skip to content
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

Fix requirements #70

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 35 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,41 @@ Mientras seguimos la migración todos son libres de colaborar con pull requests.

Para probar el bot en acción en Telegram es necesario crear un archivo `tokenz.py` que asigne a la variable `token` el token del bot con el que se desea hacer las pruebas.

Para instalar las dependecias correr lo siguiente:
Para instalar las dependencias correr lo siguiente:
Copy link
Member

Choose a reason for hiding this comment

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

Thx

```bash
$ pip3 install -r requirements.txt
```

pero te recomendamos que mejor te armes un ambiente virtual (no es necesario hacerlo cada vez):
```bash
$ python3 -m venv venv
$ venv/bin/pip3 install -r requirements.txt
Comment on lines +16 to +17
Copy link
Member

Choose a reason for hiding this comment

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

No va un source venv/bin/activate (o activate.zsh o activate.fish) en el medio?

```

para que el bot tenga los comandos hay que instalarlos en la base de datos:
```bash
$ venv/bin/python3 install.py
```
y ahora si es posible correr el bot:
```bash
$ venv/bin/python3 dcubabot.py
```

## notas:
- la librería pony no funciona con python3.11, así que vas a necesitar instalarte python3.10
tenés muchas alternativas, descargartelo de la web, usar el paquete de tu distro, etc
Comment on lines +20 to +31
Copy link
Member

Choose a reason for hiding this comment

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

muchas muchas muchas gracias por esto <3

Otra es usar **guix** que permite además crearte todo el ambiente y tener una shell contenerizada, para instalarlo podés ver el primer link de [la guia de instalación binaria](https://guix.gnu.org/manual/en/html_node/Binary-Installation.html) que apunta a un script que automatiza el proceso.
Luego:
```bash
guix shell python3.10.7
```
y luego seguir con la instalación del ambiente virtual.

también podés correr dicha shell en un container, pero en ese caso tenés que instalar más paquetes:
```bash
guix shell -CN coreutils tzdata [email protected]
```
'-C' crea el container y '-N' habilita el uso de la red.
tené en cuenta que vas a tener que recrear el ambiente virtual en estes caso.
Comment on lines +32 to +44
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Otra es usar **guix** que permite además crearte todo el ambiente y tener una shell contenerizada, para instalarlo podés ver el primer link de [la guia de instalación binaria](https://guix.gnu.org/manual/en/html_node/Binary-Installation.html) que apunta a un script que automatiza el proceso.
Luego:
```bash
guix shell python3.10.7
```
y luego seguir con la instalación del ambiente virtual.
también podés correr dicha shell en un container, pero en ese caso tenés que instalar más paquetes:
```bash
guix shell -CN coreutils tzdata [email protected]
```
'-C' crea el container y '-N' habilita el uso de la red.
tené en cuenta que vas a tener que recrear el ambiente virtual en estes caso.

Banco que guix está piola pero no es el lugar para hacerle publicidad imho



2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
pony
python_telegram_bot
python_telegram_bot==13.5
Copy link
Member

Choose a reason for hiding this comment

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

El pin de versión es importante?

icalevents
pytz
requests
Expand Down