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

Change l'approche des auteurs #2687

Merged
merged 1 commit into from
May 14, 2015
Merged

Change l'approche des auteurs #2687

merged 1 commit into from
May 14, 2015

Conversation

Eskimon
Copy link
Contributor

@Eskimon Eskimon commented May 13, 2015

Q R
Correction de bugs ? pas vraiment
Nouvelle Fonctionnalité ? non
Tickets (issues) concernés #2659

J'ai change l'approche pour le fichier AUTHORS. Plutôt que de fournir une liste a maintenir, je fournis un lien vers la liste des gens faisant parti de l’équipe dans le dépôt. Ca me semble aussi juste et plus besoin de maintenance.

@pierre-24
Copy link
Member

Comme j'ai dit, le problème, c'est que cette page fait une croix sur les développeurs passés ;)

@Eskimon
Copy link
Contributor Author

Eskimon commented May 13, 2015

mieux ?

@GerardPaligot
Copy link
Member

mieux ?

C'est pas exactement la même chose ?

Sinon, à quoi sert ce fichier ? Pourquoi GitHub le recommande ? Est-ce qu'il le recommande ?

@pierre-24
Copy link
Member

Mieux :)

Ceci étant dit, avant de merger, je vais quand même attendre d'autres avis.

@Eskimon
Copy link
Contributor Author

Eskimon commented May 13, 2015

C'est pas exactement la même chose ?

Nan. Ma contrib précédente montrait les contribs que l'on a validé manuellement comme membre de l'orga alors que celui-ci c'est "tout ceux qui ont fait au moins un commit"

@Eskimon
Copy link
Contributor Author

Eskimon commented May 13, 2015

Ca marche bien mieux quand on add le fichier après l'avoir modifié...

@SpaceFox
Copy link
Contributor

OK pour moi.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 38b987b on Eskimon:fix-2659 into 85e62f9 on zestedesavoir:release-v15.5.1.

1 similar comment
@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 38b987b on Eskimon:fix-2659 into 85e62f9 on zestedesavoir:release-v15.5.1.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 81.9% when pulling 38b987b on Eskimon:fix-2659 into 85e62f9 on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.91% when pulling 38b987b on Eskimon:fix-2659 into 85e62f9 on zestedesavoir:dev.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling b1ad61d on Eskimon:fix-2659 into 2472280 on zestedesavoir:dev.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.9% when pulling b1ad61d on Eskimon:fix-2659 into 2472280 on zestedesavoir:dev.

@Eskimon
Copy link
Contributor Author

Eskimon commented May 14, 2015

Bon pour merge du coup je pense (c'est un peu le boxon avec mes edit et mon rebase mais ca roule pour Travis maintenant)

@Eskimon
Copy link
Contributor Author

Eskimon commented May 14, 2015

@Situphen tu as vu ici tout passe bien pourtant c'est la meme architecture (travis+docker+font sur google) que l'autre...

@pierre-24
Copy link
Member

Si c'est bon, allons-y :)

pierre-24 added a commit that referenced this pull request May 14, 2015
Change l'approche des auteurs
@pierre-24 pierre-24 merged commit 1b92ed9 into zestedesavoir:dev May 14, 2015
@pierre-24 pierre-24 added this to the Version de développement milestone May 14, 2015
@pierre-24 pierre-24 added Evolution Facile Bon ticket pour débuter pour rejoindre le développement ! labels May 14, 2015
@Eskimon Eskimon deleted the fix-2659 branch May 15, 2015 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Facile Bon ticket pour débuter pour rejoindre le développement !
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants