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

Check repos #75

Merged
merged 6 commits into from
Nov 6, 2024
Merged

Check repos #75

merged 6 commits into from
Nov 6, 2024

Conversation

samuelemusiani
Copy link
Member

No description provided.

@samuelemusiani
Copy link
Member Author

@foxyseta la action di cui parlavamo per controllare quali corsi sono stati tolti. Ho rimosso quelli che restituiscono 404

@foxyseta
Copy link
Member

Se vedo bene sono tutte ing- a parte una, che è ingegneria del software orientata ai servizi.

@cartabinaria/ingegneria-informatica queste repo mancanti dovrebbero esistere? Sono cose che non abbiamo mai avuto il tempo di creare? Hanno un nome sbagliato ma esistoni?

@gabrielegenovese se ricordo bene ingegneria el software orientata ai servizi è qualcosa che abbiamo rinominato insieme dandole un altro nome? Se sì, il nome nuovo è ancora su config?

@jgabaut
Copy link
Member

jgabaut commented Nov 1, 2024

@cartabinaria/ingegneria-informatica queste repo mancanti dovrebbero esistere? Sono cose che non abbiamo mai avuto il tempo di creare? Hanno un nome sbagliato ma esistoni?

Mi scuso in anticipo se la mia risposta non sarà particolarmente utile!

In ordine:

  • Dovrebbero esistere? Non saprei, immagino che sarebbe utile avere materiale anche per questi corsi quanto per gli altri, ma forse non ho ben interpretato la domanda
  • Non abbiamo mai avuto il tempo di creare? Probabilmente si, credo
  • Hanno un nome sbagliato ma esistono? Nessuna delle due, i nomi rimossi mi sembrano di corsi obbligatori (*) ma non mi sembra ci sia già un repo corrispondente

(*) Nota per:

  • Ingegneria del software orientata ai servizi: Confermo che non credo sia un corso di ing
  • Diritto dell'Informatica: A differenza dei restanti non è obbligatorio (precisazione inutile probabilmente)

@foxyseta
Copy link
Member

foxyseta commented Nov 1, 2024

Grazie mille!

Ingegneria del software orientata ai servizi: Confermo che non credo sia un corso di ing

Sì esatto è della nostra magistrale e se ne è occuoato geno se non ricordo male.

@samuelemusiani immagino @VaiTon avesse creato questi insegnamenti in attesa di avere effettivo materiale per creare una nuova repo. In qiesto modo Dynamik mostra comunque i bottoni (ma non sono cliccabili). Io semplicemente creerei gli insegnamenti, almeno per quelli di ingegneria.

@samuelemusiani
Copy link
Member Author

Per me va bene. Se qualcuno ha voglia può creare questi insegnamenti e appena sono online rilancio la action e controllo che effettivamente esistano. Poi modifico la PR

@gabrielegenovese
Copy link
Member

@gabrielegenovese se ricordo bene ingegneria el software orientata ai servizi è qualcosa che abbiamo rinominato insieme dandole un altro nome? Se sì, il nome nuovo è ancora su config?

Si, il nuovo nome è "Archietture Software a Microservizi" e mi sembra ci sia su config.

@foxyseta
Copy link
Member

foxyseta commented Nov 1, 2024

Esatto per me non avevamo fatto caso al fatto che fosse già stato aggiunto a config PRIMA della rinominazione, quindi abbiamo aggiunto il nome nuovo anziché sostituirlo al vecchio.

Per le repo di ingegneria potrei aver tempo io. @jgabaut su https://dynamik.vercel.app/dash/ingegneria, se includi anche i pulsanti non cliccabili, mancano insegnamenti?

@VaiTon
Copy link
Member

VaiTon commented Nov 1, 2024

@samuelemusiani immagino @VaiTon avesse creato questi insegnamenti in attesa di avere effettivo materiale per creare una nuova repo. In qiesto modo Dynamik mostra comunque i bottoni

Esatto

@samuelemusiani
Copy link
Member Author

Il problema è che su cartabinaria/update ci basiamo su questo file per sapere su quali repo eseguire lo script. Essendo che alcune di queste non esistono lo script muore a metà e richiede un intervento manuale. O creiamo queste repo e le lasciamo come il template visto che non hanno file oppure le rimuoviamo da teachings.json. A sto punto creiamole e lasciamole vuote, non so

@VaiTon
Copy link
Member

VaiTon commented Nov 1, 2024

Si, direi che possiamo crearle e lasciarle vuote

@foxyseta
Copy link
Member

foxyseta commented Nov 1, 2024

Dovrei aver tempo di lavorarci io.

In generale, mi pare che ci siamo benefici anche al di là del semplice script update nell'avere una singola sorgente di verità che ci dice cose è un insegnamento fatto e finito (con repo e tutto) e cosa no

@jgabaut
Copy link
Member

jgabaut commented Nov 1, 2024

@jgabaut su https://dynamik.vercel.app/dash/ingegneria, se includi anche i pulsanti non cliccabili, mancano insegnamenti?

Mi sembra che solo per questi corsi facoltativi "consigliati" manchi del tutto il tasto:

  • Affidabilità e Controllo della Qualità
  • Progettazione di Applicazioni Web

Immagino ce ne siano altri non tra quelli "consigliati".

@VaiTon
Copy link
Member

VaiTon commented Nov 1, 2024

@samuelemusiani ti consiglio di usare le annotazioni dei workflow GH al posto di scrittura semplice su stdout

https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/workflow-commands-for-github-actions#setting-an-error-message

@foxyseta
Copy link
Member

foxyseta commented Nov 1, 2024

  • Affidabilità e Controllo della Qualità

  • Progettazione di Applicazioni Web

Grazie! Hai i link alle chat? Perché magari potrei fare le repo anche per questi due e aggiungerli alla PR di samu

@foxyseta foxyseta added the enhancement New feature or request label Nov 1, 2024
@jgabaut
Copy link
Member

jgabaut commented Nov 1, 2024

Grazie! Hai i link alle chat? Perché magari potrei fare le repo anche per questi due e aggiungerli alla PR di samu

Mi dispiace ma no, non ho link alle eventuali chat su questi corsi. Mi dispiace! Non penso di poter aiutare diversamente per questi.

@foxyseta
Copy link
Member

foxyseta commented Nov 1, 2024

Grazie! Hai i link alle chat? Perché magari potrei fare le repo anche per questi due e aggiungerli alla PR di samu

Mi dispiace ma no, non ho link alle eventuali chat su questi corsi. Mi dispiace! Non penso di poter aiutare diversamente per questi.

Nessun problema! @VaiTon le hai tu? O proprio non esistono e volete crearle?

@VaiTon
Copy link
Member

VaiTon commented Nov 1, 2024

@foxyseta check Matrix

Copy link
Member

@foxyseta foxyseta left a comment

Choose a reason for hiding this comment

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

@jgabaut @VaiTon aggiunti "Affidabilità e controllo della qualità" e "Progettazione di applicazioni web". Aggiunte le repo per tutti i corsi di ingegneria triennali. Se alcuni non sono già cliccabili su dynamik è perché ho corretto qualche nome degli insegnamenti, quindi lo saranno quando aggiorneremo config.

@gabrielegenovese rimosso il vecchio "Ingegneria del software orientata ai servizi"

@samuelemusiani dal mio punto di vista sono pronto ad accettare la PR. Ho modificato anche degrees.json in modo appropriato. Quando te la senti accetta in main pure da solo.

@samuelemusiani
Copy link
Member Author

Magari prima di accettare completamente guardo a quella cosa per gli errori che mi ha linkato @VaiTon

@foxyseta
Copy link
Member

foxyseta commented Nov 4, 2024

@samuelemusiani ti consiglio di usare le annotazioni dei workflow GH al posto di scrittura semplice su stdout

https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/workflow-commands-for-github-actions#setting-an-error-message

Ah questa dici sì buona idea

@samuelemusiani samuelemusiani merged commit 636e732 into main Nov 6, 2024
1 check passed
@samuelemusiani samuelemusiani deleted the check-repos branch November 6, 2024 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants