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

Un badge pour la santé du code #2173

Merged
merged 1 commit into from Jan 29, 2015

Conversation

@firm1
Copy link
Contributor

commented Jan 29, 2015

Q R
Correction de bugs ? non
Nouvelle Fonctionnalité ? non
Tickets concernés Aucun

Landscape fourni un badge qui analyse la santé du code et revèle ses points de faiblesse (et donc ce qu'on peut corriger). Actuellement nous sommes à 87%.

On pourra donc juger d'une mauvaise PR assez facilement (si elle fait décroitre la santé du code).

PS : il ne traite que le code python.

QA : pas grand chose à faire.

Instruction pour DTC : rajouter le service landscape dans les settings du projet github.

@SpaceFox

This comment has been minimized.

Copy link
Member

commented Jan 29, 2015

On pourra donc juger d'une mauvaise PR assez facilement (si elle fait décroitre la santé du code).

Ça alerte dans la PR (à la Coverall) ou ça se contente de changer l'aspect d'un badge ?

@@ -1,10 +1,12 @@
[![Build Status](https://travis-ci.org/zestedesavoir/zds-site.svg?branch=dev)](https://travis-ci.org/zestedesavoir/zds-site)
[![Coverage Status](https://coveralls.io/repos/zestedesavoir/zds-site/badge.png?branch=dev)](https://coveralls.io/r/zestedesavoir/zds-site?branch=dev)
[![Licence GPL](http://img.shields.io/badge/license-GPL-yellow.svg)](http://www.gnu.org/licenses/quick-guide-gplv3.fr.html)

This comment has been minimized.

Copy link
@Eskimon

Eskimon Jan 29, 2015

Member

Juste pour clarifier:

  • Tu vires le badge de la licence car il sert pas a grand chose ?
  • Tu vires le badge de la doc car il y a un hook pour s'assurer qu'elle est toujours a jour ?

This comment has been minimized.

Copy link
@Eskimon

Eskimon Jan 29, 2015

Member

Autant pour moi, j'avais pas bien fait attention et je vient de voir que tout ca est parti en bas...

This comment has been minimized.

Copy link
@SpaceFox

SpaceFox Jan 29, 2015

Member

XD j'ai failli faire le même commentaire avant de me rendre compte que ça avait été déplacé !

@firm1

This comment has been minimized.

Copy link
Contributor Author

commented Jan 29, 2015

Ça alerte dans la PR (à la Coverall) ou ça se contente de changer l'aspect d'un badge ?

Juste le badge je crois. Les alertes à la coveralls sont en experimental

@SpaceFox

This comment has been minimized.

Copy link
Member

commented Jan 29, 2015

Alors pourquoi pas, mais ça ne me paraît pas super utile du coup : personne ne pensera jamais à vérifier la modification apportée sur ce badge pour une PR en particulier.

@firm1

This comment has been minimized.

Copy link
Contributor Author

commented Jan 29, 2015

Ouaip pas faux. Après pour bien faire il faudrait reprendre la PR de @steenzout avec prospector pour integrer landscape

@GerardPaligot

This comment has been minimized.

Copy link
Member

commented Jan 29, 2015

@firm1

This comment has been minimized.

Copy link
Contributor Author

commented Jan 29, 2015

Yeah. Je merge du coup.

firm1 added a commit that referenced this pull request Jan 29, 2015
Merge pull request #2173 from firm1/add_landscape
Un badge pour la santé du code

@firm1 firm1 merged commit c1f58b3 into zestedesavoir:dev Jan 29, 2015

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@steenzout

This comment has been minimized.

Copy link
Contributor

commented Jan 29, 2015

as a note, usually projects put their badges below their project name ;)
you may to consider doing the same.

@SpaceFox SpaceFox added this to the Version 1.6 milestone Jan 29, 2015

@SpaceFox

This comment has been minimized.

Copy link
Member

commented Jan 29, 2015

Instruction pour DTC : rajouter le service landscape dans les settings du projet github.

J'ai rien fait mais ça a l'air de marcher. J'ai encore quelque chose à faire ici ?

@firm1

This comment has been minimized.

Copy link
Contributor Author

commented Jan 29, 2015

as a note, usually projects put their badges below their project name ;)
you may to consider doing the same.

I think so.

J'ai rien fait mais ça a l'air de marcher. J'ai encore quelque chose à faire ici ?

Je trouve ça bizarre quand même mais si c'est bon, tu peux laisser tomber.

@firm1 firm1 deleted the firm1:add_landscape branch Jun 24, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.