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

Adicionar arquivo de configuração do scrutinizer #233

Closed
MarceloCajueiro opened this issue Jun 12, 2018 · 7 comments
Closed

Adicionar arquivo de configuração do scrutinizer #233

MarceloCajueiro opened this issue Jun 12, 2018 · 7 comments
Labels
good first issue Contribuição fácil de fazer

Comments

@MarceloCajueiro
Copy link
Contributor

MarceloCajueiro commented Jun 12, 2018

Objetivo: fazer o serviço rodar da melhor forma possível para o projeto.

Referência para quem for fazer: #211 (review)

Link do scrutinizer: https://scrutinizer-ci.com/g/portabilis/i-educar/inspections/396936a9-37ad-42df-a81c-804eaac68f78

@MarceloCajueiro MarceloCajueiro added the good first issue Contribuição fácil de fazer label Jun 12, 2018
@farribeiro
Copy link
Contributor

farribeiro commented Jun 12, 2018

eu posso fazer um cherry-pick dos commits do @williamespindola e abrir um PR daquelas outras alterações

Autoria continua mantida!

@MarceloCajueiro
Copy link
Contributor Author

Acho Ok, mas tem que ser commits separados. Cada PR resolvendo um problema.

@farribeiro
Copy link
Contributor

Isso não é necessário, olhe para seus branches cheios de commiters variados

@williamespindola
Copy link
Contributor

Da para ficar apenas um commit, basta fazer amend no meu commit. Manda bala @farribeiro se precisar de ajuda da um grito.

@williamespindola
Copy link
Contributor

Da para usar o do pr #211 tranquilamente, basta adicionar o trecho que o Cajueiro falou:

- 'ieducar/tests/*'
- 'ieducar/misc/**/*'

e @MarceloCajueiro não tem problema eles ficaram red até estarem passando. Isto é bom naverdade pois da para ver onde precisamos atacar.

@williamespindola
Copy link
Contributor

Se não quiser pegar do PR pode copiar daqui também

filter:
    excluded_paths:
        - tests/*

checks:
    php:
        code_rating: true

tools:
    external_code_coverage: true
    php_analyzer: true
    php_changetracking: true
    php_code_sniffer:
        config:
            standard: "PSR2"
    php_cpd: true
    php_mess_detector: true
    php_pdepend: true
    sensiolabs_security_checker: true

@farribeiro
Copy link
Contributor

farribeiro commented Jun 13, 2018

Tem que extraír o Contribuiting.md é o readme.md ... como estou fora do ambiente de desenvolvimento... vou colar conforme solicitado pelo desktop pessoal

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Contribuição fácil de fazer
Projects
None yet
Development

No branches or pull requests

3 participants