Come contribuire.
Lo sviluppo di DXPR Builder avviene su GitHub all’indirizzo github.com/dxpr/dxpr_builder. Tutti i contributi seguono il flusso di lavoro descritto nel CONTRIBUTING.md del repository.
Flusso di lavoro Git
- Ogni pull request deve essere collegata a una issue di GitHub
- Convenzione per i nomi dei branch:
person/target-branch/#issue-description(ad es.jur/3.x/#42-fix-carousel-autoplay) - Scrivi commit atomici seguendo lo styleguide Git di alphagov
- Le pull request devono contenere una breve descrizione della soluzione
- Mantieni le modifiche entro l’ambito della issue: niente formattazione o refactoring non correlati
Code owner
@jjroelofs è il code owner. Nessuna pull request può essere unita senza la sua revisione.
Standard di programmazione
- PHP: standard di programmazione Drupal
- JavaScript: solo ES6+ (niente
var, niente jQuery nel nuovo codice). Segui gli standard di programmazione JavaScript di Airbnb con le eccezioni del progetto - CSS: segui il Design System per le proprietà personalizzate e i nomi dei token
- Una modifica al metodo
showeddi un elemento deve includere il filedxpr_frontend.jsaggiornato. Genera il file dall’output completo che l’editor stampa nella console del browser quando fai clic sul pulsante di salvataggio di un contenitore. - Le nuove funzioni richiedono la copertura di QA Analytics.
Esegui npm install una volta dopo la clonazione. Installa l’hook
pre-commit di Husky, che controlla con il linter i file che hai messo nello stage; vedi
Test.
Esecuzione dei test E2E
I test si avviano con comandi slash nei commenti delle pull request:
| Comando | Descrizione |
|---|---|
/e2e |
Esegue le suite complete dell’editor BS5 e per utenti anonimi |
/e2e groups=group-name |
Esegue gruppi di test specifici |
/e2e tests=ClassName |
Esegue classi di test specifiche |
/e2e-bs3 |
Esegue la suite dell’editor BS3 |
/e2eV2 |
Esegue le suite BS5 sullo stack DXPR CMS |
/e2e maven_ref=branch |
Esegue i test con un branch dxpr_maven personalizzato |
Quando una modifica del builder richiede una modifica dei test, l’ingegnere QA aggiorna
dxpr_maven su un branch ed esegui /e2e maven_ref=<branch> sulla
tua pull request. Prima viene unita la pull request del builder, poi quella di
dxpr_maven.
Prima di richiedere la revisione
- Implementa la modifica
- Testa con attenzione
- Fai un giro di pulizia e refactoring
- Ripeti i test per confermare che niente si sia rotto
- Fai commit e push con una cronologia git pulita
- Verifica che i controlli del linter riescano
- Commenta
/e2esulla pull request per eseguire i test di regressione - Richiedi la revisione dopo che tutti i controlli sono riusciti