outil prospector

[steenzout]

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

comme flake8 marche pas très bien,
comme vous utilisez pas tox,
comme vous utilisez pas landscape.io pour determiner la qualité de votre code,
voici un petit contribution qui pourras aider a quiconque verifier la qualité du code actuel tel comme landscape.io le fait.

$ prospector -p zds

ca prends du temps mais ca vale la peine.
vous avez pas besoin de faire un merge avec dev.
vous pouvez très facilment tester votre branch de la facon suivante:

$ git checkout prospector
$ git checkout -b propector_issue_xyz
$ git checkout issue_xyz
(tout les changements de code)
$ git checkout propector_issue_xyz
$ git merge xyz
$ prospector -p zds

le seul inconvenient c'est de avoir besoin de changer pour le branch issue_xyz, faire les changements, checkout prospector_issue_xyz, merge, prospector a chaque fois.

vous pouvez aussi dire que vous utiliserais prospector seulement pour dev (sans le mettre comme depedance) et créer un prospector_dev dans le quel vous ferais toujour merge avec dev pour verifier le code de votre application.

dernier alternative c'est d'utiliser, tout simplement landscape.io (gratuit).
voici le lien de la qualité de code des branches que j'ai sur mon compte github.

https://landscape.io/github/steenzout/zds-site/

[gustavi]

@steenzout encore une fois : ceci nécessite en premier lieu une discussion avec l'équipe de développement sur le forum. C'est une bonne initiative mais il FAUT en parler avant. https://zestedesavoir.com/forums/communaute/dev-zone/

[steenzout]

finalment, quelqu'un a mis un lien. :)

[steenzout]

vous et la politique :)

j'aurais pu faire pas mal de lignes de code sur d'autres projects mais non, il fallait ecrire en francais et comprendre votre processus.
j'etait surpris!
vous etes si nombreux ou avec beaucoup des visites pour avoir besoin de tout ca?
alexa.com peut-etre marche pas si bien pour l'Europe...

voila, lien sur le sujet sur dev zone

@gustavi , _satisfait_? :)

t'enquietes pas celui-ci sera ma dernière proposition sur github (quel sacrilege!!!!). lol

[poulp]

vous etes si nombreux ou avec beaucoup des visites pour avoir besoin de tout ca?

la qualité n'attend pas le nombre d'utilisateur ;)

[gustavi]

@steenzout du calme, il est inutile de s'énerver. On a une politique pour la résolution des bugs et l'ajout de nouvelles fonctionnalités, il faut la respecter. Comprend bien que si chacun arrive avec sa PR qui rajoute un outil (même si c'est justifié) sans aucune discussion au préalable c'est pas gérable.

Plusieurs choses à te faire remarquer (et à ne pas mal prendre, ce n'est pas le but) :

• tu fais une PR avec l'ajout d'un outil sans discussion préalable et sans étude des alternatives (si jamais il y en a). Le forum est là pour ça.
• le build ne passe pas ;
• tous les projets sérieux ont un processus spécifique pour ce qui est gestion des tickets et des PR, surtout quand il y a une grosse activité (ce qui est plutôt le case chez nous http://zestedesavoir.com/articles/99/zds-passe-en-version-11-quelques-jours-avant-son-anniversaire/ ).

Ta contribution est la bienvenue. Cependant il faut voir GitHub comme un outil de travail et non pas comme un outil pour les propositions, même bien fondées.

[steenzout]

build

@gustavi , le build marche pas parce que prospector montre tout ce que était cacher.
est-que tu as vu ce qui travis a rapporte?

je peut change le config pour cacher de nouveau.
c'est ca que tu veux?

les erreurs sont "zds/gallery/tests/tests_models.py:15:9: N802 function name should be lowercase".
est-ce que tu veux que change le code pour suivre une politique renforce par moi ou vous avez besoin de discuter aussi sur dev?

j'ai des outils qui font changer le nom des functions sans problème.
donnez moi votre avis et je ferais une autre contribution pour corriger ce build si
non après que vous aurez "accepte" ou pas le commit et decider les règles de sintaxe de votre project,
je pourrais corriger tout les problèmes.

2

c'est pas parce que je fait une contribution que vous etes obliges de l'accepter.
je fesais pas parti de votre communaute donc c'était pas si dificille de:

• merci steenzout, on discutera (un jour) l'integration de ton contribution et on vera: no promises
• si vous croyais que c'était peut-être un bonne contribution, un entre vous pourrais faire un post sur votre communaute et dire: voila, steenzout a poste un contribution a github. qu'est que vous dites? et me laisser dehors de la politique
• après votre decision (quelques jours, semaines, mois, années?), c'était a vous d'intègre ou pas le commit.

j'aurais jamais aucun probleme (comme j'ai deja dit) n'importe quel decision vous prendrais.

j'était pas de tout enerve si non j'aurais meme pas m'enregistre sur le site et j'auras pas pris le temps d'ecrire en francais la proposition.
j'averti que je suis pas francophone donc ma façon d'ecrire n'exprimeras jamais d'une façon correct mes sentiments ou les idées que j'aimerais transmettre.
pour cela aussi j'aurais aime être dehors vos discussions internes.
preuve de cela c'est cette thread.

[steenzout]

@poulp mais la "bureaucracy" devrait attendre un plus grand nombre d'utilisateurs, non? ;)

just guessing...

[Eskimon]

@steenzout toutes les propositions sont le bienvenues, ca c'est sur. Mais il faut se mettre aussi à la place des contributeurs réguliers. Tu arrives de nulle part en faisant des propositions d'autorité en disant "il faut faire comme ca", sans expliquer pourquoi, quel sera l'impact etc... Peut-être que c'est trop de bureaucratie, mais comme on veut que tout le monde participe il faut aussi comprendre ce que chaque chose font.

Les remarques qui ont été faite ne sont pas des reproches mais des invitations à la discussion :)

Enfin, pour le côté francophone bah... On pouvait pas deviner :D

En tout cas merci de tes contributions !

[GerardPaligot]

@steenzout First, thanks for your contributions. I am not intervened in your issues but all of them are relevants (this one too). I don't want to say what others have already said but I'm curious. You are from Salt Lake City, right? How do you find the project "Zeste de Savoir"?

Pour les autres: Pour moi, si une personne ne parle pas le français, nous ne devons pas être fermé à l'anglais. Invitez les à parler l'anglais. Ils pourront s'exprimer convenablement et mieux détailler leurs contributions. Evitez aussi d'être trop agressif. @gustavi, désolé mais je t'ai vraiment trouvé agressif sur la plupart des issues de @steenzout.

PS : Du coup, je commence vraiment à me dire qu'il faut réécrire la documentation en anglais. Cela devient nécessaire.

[gustavi]

@GerardPaligot ce n'était pas voulu. Passons.

https://zestedesavoir.com/forums/sujet/1860/traduction-du-la-doc-et-des-zep/

[steenzout]

@GerardPaligot , I know different cultures express themselves in different ways and I'm used to it so no offense taken.

I found the project through 24pullrequests.com (1 open source contribution a day till December 24).
I'm trying to make 1 contribution a day for the 24pullrequests project.
I lived in France for 4 years so seeing you were doing Python and were French it caught my attention and I just tried to push some contributions.
Since I started late, I guess with the rush to catchup I've submitted a lit bit too many contributions at a time for the project.

yes, I'm in SLC, UT.
if you have a LinkedIn account you guys can send me an invite.

[Eskimon]

A fermer pour utiliser directement l'outil landscape.io comme proposé plus tot plutot que d'alourdir la stack d'outil ? (in english if needed :) )

[SpaceFox]

Is there something to keep here or was it just a PR for the 24pullrequests thing?

[steenzout]

there are several issues with your code base today (cyclic dependencies is a blatant one) and it's clear landscape.io integration or the use of prospector when you run your tests would useful if you look into its report.

[SpaceFox]

On a maintenant landscape.io. Donc cette PR me semble devenue inutile.