Refactor del codice per i vari tipi di deploy #2

Open
opened 9 months ago by aziis98 · 0 comments
Owner

Non mi piace molto com'è organizzato il codice di jobs.ts, runners.ts e deploys.ts, vorrei tenerlo abbastanza generico in modo da magari scrivere qualche test prima o poi nella vita però boh.

Alcune delle richieste sono:

  • Avere delle funzioni (pure sperabilmente) che si occupano di fare i deploy, chiamare docker o exec e loggare un po' di informazioni utili tipo i comandi/azioni che stanno succedendo.

  • Per ora c'è il concetto di Runner che dovrebbe rappresentare un modo per eseguire comandi senza chiamare direttamente exec. Però ora ad esempio docker è chiamato direttamente e non mi piace il modo di streammare i log (che serve per poter vedere le cose live lato client)

  • Il nuovo file events.ts invece sembra promettente.

Idee e/o proposte su come aggiustare tutto ciò sono ben accette.

Non mi piace molto com'è organizzato il codice di [jobs.ts](src/branch/main/src/jobs.ts), [runners.ts](src/branch/main/src/runners.ts) e [deploys.ts](src/branch/main/src/deploys.ts), vorrei tenerlo abbastanza generico in modo da magari scrivere qualche test prima o poi nella vita però boh. Alcune delle richieste sono: - Avere delle funzioni ([pure](https://en.wikipedia.org/wiki/Pure_function) sperabilmente) che si occupano di fare i deploy, chiamare docker o exec e loggare un po' di informazioni utili tipo i comandi/azioni che stanno succedendo. - Per ora c'è il concetto di `Runner` che dovrebbe rappresentare un modo per eseguire comandi senza chiamare direttamente `exec`. Però ora ad esempio docker è chiamato direttamente e non mi piace il modo di streammare i log (che serve per poter vedere le cose live lato client) - Il nuovo file events.ts invece sembra promettente. Idee e/o proposte su come aggiustare tutto ciò sono ben accette.
Sign in to join this conversation.
No Label
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: phc/phcd#2
Loading…
There is no content yet.