csunibo / config

Una raccolta centralizzata delle configurazioni per CSUnibo.
https://csunibo.students.cs.unibo.it/wiki/infrastruttura/configurazioni/index.html
GNU Affero General Public License v3.0
0 stars 3 forks source link

Integrare README.md in componenti svelte #45

Closed ali-benny closed 4 months ago

ali-benny commented 1 year ago

In ogni pagina dedicata ad un insegnamento, invece di avere il readme.md come file, non avrebbe più senso inserirlo nella pagina come componente?

foxyseta commented 1 year ago

A dirla tutta, nelle repository degli insegnamenti recenti, (ad esempio https://github.com/csunibo/intelligenza-artificiale), il README.md è solo un rimando alla wiki. Forse potremmo direttamente escluderlo dall'elenco dei file, lasciando visibile solo LICENSE.txt. Non so.

foxyseta commented 5 months ago

@ali-benny quindi o lo nascondi (come impostazione lato static opt-in che accendiamo in dynamik?) o lascimao tutto così

samuelemusiani commented 5 months ago

Ma se lo togliessimo da statik? Tanto l'opzione per escludere una regex la ha

foxyseta commented 5 months ago

Ci avevo pensato, ma bisogna farlo in un posto che influenzi tutte le funzionalità di dynamik presenti (e.g. la ricerca fuzzy) e future. Perciò cambiare lato dynamik sicuro è più veloce da scrivere ma forse non pulito pulito concettualmente.

Se pensate sia comunque meglio dynamik allora @ali-benny può fare da lì.

samuelemusiani commented 5 months ago

La cosa più pulita in assoluto direi che è cambiare lato statik. @ali-benny te che dici?

foxyseta commented 5 months ago

Ah oddio avevo letto male il tuo post precedente ahahahah. Sìsì ero d'accordo su fare lato static ma la mettrei come opzione opt-in giusto per preservare la retrocompatibilità di statik.

ali-benny commented 5 months ago

La cosa più pulita in assoluto direi che è cambiare lato statik. @ali-benny te che dici?

Scusatemi, ho letto solo ora. Facciamo come preferite davvero per me era inutile lasciarlo visibile come file a sé "scaricabile" o lo usiamo o lo nasconderei

foxyseta commented 5 months ago

@samuelemusiani opzione di default spenta su statik va bene anche a te?

samuelemusiani commented 5 months ago

Essendo che stati permette di passargli una regex di esclusione io metterei questa regex o nella action globale in config (non ho provato ma secondo me funziona) o lo mettiamo in ogni repo. La prima è più veloce, la seconda permette una granularità delle opzioni migliore. Andrei comunque con la prima se funziona

foxyseta commented 5 months ago

Assolutamente la prima e speriamo non ci sia mai bisogno di passare alla seconda. Mi ero dimenticato della regex. @ali-benny riesci tu?

samuelemusiani commented 4 months ago

Fatto https://github.com/csunibo/config/commit/d808df4cdeb3db7a906c9c052c430206d7dd8e76