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.

Sign up for GitHub

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

Jump to bottom

Scrutinizer score #277

Merged
merged 4 commits into from
Aug 14, 2018
Merged

Scrutinizer score #277

merged 4 commits into from
Aug 14, 2018

Conversation

farribeiro
Copy link
Contributor

@farribeiro farribeiro commented Jun 23, 2018 •
edited
Loading

Badge oferecido pelo scrutinizer... informativo e enriquece o documento, não cosmético, seguindo tendências de outros projetos FOSS

Link #271

fernandosjp
fernandosjp approved these changes Jun 29, 2018
View reviewed changes
Copy link
Contributor

@fernandosjp fernandosjp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Acho bem legal ter os badges. Já funciona de atalho para o Scrutinizer. Só de sugestão, talvez colocaria até os outros badges do scrutnizer (build e code coverage no futuro). O que acha @farribeiro?

Latest Release
Build Status
Scrutinizer Code Quality
Code Coverage

@farribeiro
Copy link
Contributor Author

farribeiro commented Jun 29, 2018 •
edited
Loading

Vou analisar como aplica outros badges, ASAP. Me ocupei recentemente

@fernandosjp
Copy link
Contributor

Pode pegar os códigos que eu usei no comentario que já são para o repositório do i-Educar.

@farribeiro
Copy link
Contributor Author

farribeiro commented Jun 29, 2018 •
edited
Loading

Daqui (dos comentários) não se retira o badge... ASAP farei adaptações

Lembrando que estes badges são hotlinks para o scrutinizer

farribeiro
farribeiro commented Jul 1, 2018
View reviewed changes
Copy link
Contributor Author

@farribeiro farribeiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solicito review

@fernandosjp
Copy link
Contributor

fernandosjp commented Jul 2, 2018 •
edited
Loading

@farribeiro o badge de latest release também seria legal deixar para quem visitar esporadicamente a home do projeto saber qual foi o ultimo release. :) Acho que mudaria o nome do PR para "Incluir badges no README (latest release e scrutinizer)"

@farribeiro
Copy link
Contributor Author

Eu não achei este badge... Poderia me apontar o hotlink?

@fernandosjp
Copy link
Contributor

fernandosjp commented Jul 2, 2018 •
edited
Loading

[![Latest Release](https://img.shields.io/github/release/portabilis/i-educar.svg?label=latest%20release)](https://github.com/portabilis/i-educar/releases)

@farribeiro
Copy link
Contributor Author

farribeiro commented Jul 2, 2018 •
edited
Loading

Farei no dia seguinte, estou em happy hour. ASAP

@ghost
Copy link

ghost commented Jul 2, 2018

Ainda usam raça no cadastro? O politicamente correto agora não é "etnia" ?

@farribeiro
Copy link
Contributor Author

Tá comentando fora da issue? Solicite discussão de alteração na tela

@ghost
Copy link

ghost commented Jul 2, 2018

Sorry! Desconsidere o comentário (que nem sei se é pertinente)

@farribeiro
Copy link
Contributor Author

farribeiro commented Jul 2, 2018 •
edited
Loading

Só não poluir com outros assuntos

@farribeiro
Copy link
Contributor Author

farribeiro commented Jul 3, 2018 •
edited
Loading

@fernandosjp gostei do projeto de escudo... e dá para trazer mais escudos.

Thanks... é o mesmo badge que você estava usando e estava funcionando.

farribeiro
farribeiro commented Jul 7, 2018
View reviewed changes
Copy link
Contributor Author

@farribeiro farribeiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solicito review

@farribeiro farribeiro self-assigned this Jul 11, 2018
@farribeiro farribeiro closed this Jul 12, 2018
@farribeiro farribeiro deleted the patch-1 branch July 12, 2018 14:54
@farribeiro farribeiro restored the patch-1 branch July 13, 2018 01:57
@farribeiro farribeiro reopened this Aug 10, 2018
farribeiro
farribeiro commented Aug 10, 2018
View reviewed changes
Copy link
Contributor Author

@farribeiro farribeiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solicito review

@edersoares edersoares merged commit 8968d55 into portabilis:master Aug 14, 2018
@farribeiro farribeiro deleted the patch-1 branch August 14, 2018 23:43
edersoares pushed a commit that referenced this pull request Sep 6, 2018
@munizeverton
Merge branch 'comunidade' into comunidade-patch-2018-08-24
da7204b 
* comunidade:
  Usa a versão 1.4.3 do pacote unshiftio/url-parse
  Adição dos textos na página de documentação (#373)
  Scrutinizer score (#277)
  Remove análise de arquivos JavaScript no Scrutinizer (#378)
  Removido função que causava warning
  review
  Atualiza dependências do composer
  Atualização da documentação
  Removendo instalação desnecessária do composer
  Adicionado composer ao docker-composer.yml Para remover dependência de ter de instalar o composer para depois instalar as dependências do projeto, foi adicionado ao docker-compose.yml a imagem oficia do composer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@edersoares edersoares edersoares approved these changes

@fernandosjp fernandosjp fernandosjp approved these changes

Assignees

@farribeiro farribeiro

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@farribeiro @fernandosjp @edersoares