search

pythonitalia / python-italy-telegram-bot

Modular telegram group management bot, used for the Python Italia telegram group
https://www.python.it/comunita/
GNU General Public License v3.0
16 stars 8 forks source link

Complete code review #25

Closed MattiaFailla closed 4 years ago

MattiaFailla commented 5 years ago

Con questa PR speriamo di portare a termine il lavoro sul bot. Principalmente dobbiamo fare il review di tutto il codice alla ricerca di eventuali imperfezioni. Inoltre, dove possibile, sarebbe utile aggiungere dei commenti più esplicativi su come si comporta il codice (vedesi fase di mute all'entrata di un nuovo utente).

hearot commented 5 years ago

Appena avrò il tempo necessario, mi metterò a guardare con precisione tutto! :v:

MattiaFailla commented 4 years ago

Altra review parziale. Comunque complimenti a chi ha fatto le traduzioni: non vi siete fatti scappare nulla.

Grazie hehe

MattiaFailla commented 4 years ago

Credo che sia ora di portare a termine la review. Attivo la review per il merge.

iflare3g commented 4 years ago

Per me è ok! Poi se c'è altro che possiamo migliorare/aggiungere, possiamo lavorarci in seguito L' abbiamo lasciato appeso già da troppo tempo 😂

MattiaFailla commented 4 years ago

Per me è ok! Poi se c'è altro che possiamo migliorare/aggiungere, possiamo lavorarci in seguito L' abbiamo lasciato appeso già da troppo tempo joy

Riesci a marcare la PR come reviewed?

iflare3g commented 4 years ago

Devo provare da PC che da telefono mi sa che non me lo fa fare ☹️

github-actions[bot] commented 4 years ago

Questa pull-request è inattiva da molto tempo. Per favore, controlla le nuove versioni del bot e se pensi che la PR sia ancora valida commenta qui.

Kavuti commented 4 years ago

Riapertura della pull request

MattiaFailla commented 4 years ago

@Kavuti la mergiamo? In qualsiasi caso possiamo andare a riutilizzare questo codice per altri scopi.