Contribuir.
O desenvolvimento do DXPR Builder decorre no GitHub em github.com/dxpr/dxpr_builder. Todos os contributos seguem o fluxo de trabalho descrito no CONTRIBUTING.md do repositório.
Fluxo de trabalho Git
- Cada pull request tem de estar ligado a um GitHub issue
- Convenção de nomes de ramos:
person/target-branch/#issue-description(p. ex.jur/3.x/#42-fix-carousel-autoplay) - Escreva commits atómicos segundo o guia de estilo Git da alphagov
- Cada pull request tem de conter uma descrição curta da solução
- Mantenha as alterações dentro do âmbito do issue: sem formatação nem refatoração alheias
Responsável pelo código
@jjroelofs é o responsável pelo código. Nenhum pull request pode ser fundido sem a revisão do responsável pelo código.
Normas de codificação
- PHP: normas de codificação do Drupal
- JavaScript: apenas ES6+ (sem
var, sem jQuery em código novo). Siga as normas de codificação JavaScript da Airbnb com exceções do projeto - CSS: siga o sistema de design para propriedades personalizadas e nomes de tokens
- Uma alteração ao método
showedde um elemento tem de incluir odxpr_frontend.jsatualizado. Gere-o a partir da saída completa que o editor imprime na consola do navegador quando clica no botão de guardar de um contentor. - As funcionalidades novas precisam de cobertura do QA Analytics.
Execute npm install uma vez após clonar. Instala o hook de pré-commit do Husky,
que verifica os ficheiros em staging; consulte
Testes.
Executar testes E2E
Os testes são acionados por comandos de barra em comentários de PR:
| Command | Description |
|---|---|
/e2e |
Executa as suites completas do editor BS5 e de utilizador anónimo |
/e2e groups=group-name |
Executa grupos de testes específicos |
/e2e tests=ClassName |
Executa classes de teste específicas |
/e2e-bs3 |
Executa a suite do editor BS3 |
/e2eV2 |
Executa as suites BS5 na pilha DXPR CMS |
/e2e maven_ref=branch |
Testa com um ramo dxpr_maven personalizado |
Quando uma alteração do construtor exige uma alteração de testes, o engenheiro de QA atualiza
o dxpr_maven num ramo e executa /e2e maven_ref=<branch> no
seu pull request. O pull request do construtor é fundido primeiro e depois o do
dxpr_maven.
Antes de pedir revisão
- Implemente a alteração
- Teste com cuidado
- Faça uma ronda de limpeza/refatoração
- Teste de novo para confirmar que nada partiu
- Faça commit e push com um histórico Git limpo
- Confirme que as verificações do linter passam
- Comente
/e2eno PR para executar os testes de regressão - Peça revisão depois de todas as verificações passarem