Concordo, come ho già affermato, si trattava di una prima stesura che ora è stata migliorata grazie ai vostri feedback. Abbiamo dato un’altra occhiata e avevate ragione.
Aggiunto al metodo di revisione
-
Classificazione dei candidati prima dell’ispezione dei file:
- Plugin
- Tema
- Componente di tema
- Estensione ibrida
- Repository di integrazione
- Artefatto di rilascio
-
Inventario separato per repository/ rilascio condivisi:
- README, licenza, changelog
- File di pacchetto e lockfile
- Workflow CI
- Script di installazione/Docker
- Integrazioni con servizi esterni
- Asset di rilascio generati
- Rilascio con tag/archivio
- File non tracciati e ignorati rilevanti
-
Inventario espanso per i plugin:
- Ogni file caricato, registrato o esposto da
plugin.rb config/routes.rbdb/post_migrate/- Viste, engine, validatori e middleware
- Codice frontend per admin e pubblico
- Connettori, componenti, route, servizi, template e test frontend
- Stili comuni, desktop, mobile, admin e incorporati
- Fixtures, file di supporto e test di browser/sistema
- Dipendenze Ruby, JavaScript, di sistema e di servizi esterni
.discourse-compatibility- Branch e workflow
d-compat/* - Limiti di versione di Discourse dichiarati
- Ogni file caricato, registrato o esposto da
-
Nuovo inventario per temi/componenti di tema:
about.jsonnella radice- Classificazione
component - Metadati di licenza, autore, versione e compatibilità
- Asset, schemi di colore, screenshot e impostazioni personalizzabili dichiarati
settings.ymlnella radicelocales/nella radicecommon/,desktop/emobile/- File SCSS e file di iniezione HTML supportati
javascripts/nella radiceapi-initializers/- Tutti i file
.js,.gjse.hbs stylesheets/nella radice e fogli di stile importatiassets/nella radice e tutti i riferimenti ad essi- Anteprime/screenshot
- Test e configurazione lint/build
- Metadati e branch di compatibilità
- Byte del tema imballato/esportato
-
Nuovi controlli strutturali:
- Il tipo di estensione dichiarato deve corrispondere a
about.jsone al comportamento di installazione. component: trueindica un componente di tema.component: falseo omesso indica un tema completo.- I repository ibridi ricevono tutti gli inventari applicabili.
- I file mal posizionati o inaspettati vengono indagati anziché ignorati silenziosamente.
- Le directory opzionali mancanti non sono automaticamente difetti.
- Albero di lavoro, archivio di rilascio, estensione installata, asset generati e candidato pubblico sono superfici di evidenza separate.
- Il tipo di estensione dichiarato deve corrispondere a
-
Nuova checklist per temi completi:
- Identità dei metadati
- Rendering del tema completo
- Copertura delle pagine principali
- Impostazioni, locale e asset
- JavaScript/initializer API supportati
- Comportamento reattivo, accessibilità e RTL
- Comportamento di Foundation, Horizon e incorporamento
- Interazioni con i componenti di tema
- Controlli di installazione, aggiornamento, rollback e compatibilità
La mia speranza è che il contributo della comunità migliori la Skill e che la Skill aiuti gli altri a valutare il proprio lavoro o, francamente, il lavoro degli altri prima dell’installazione, in caso di dubbi sulla qualità.
Finora siamo 2 a 2. I vostri input hanno guidato miglioramenti alla Skill e @satonotdead l’ha trovata utile. Grazie per le gentili parole e il supporto @satonotdead