InnerSourceCommons / InnerSourceCommons/InnerSourceLearningPath
Pulling contributors in as reviewers
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 77
- Forks
- 47
- Merge moyen
- 11 h 11 min
- PR mergées (30 j)
- 5
Description
In the learning path for the Trusted Committer (part 2) we explain how it's the trusted committers that are reviewing code. In a first approximation this is correct and matches what I see in many open source projects.
When it comes to recruiting new committers and onboarding them I have seen a different approach:
https://blogs.apache.org/comdev/entry/an-approach-to-community-building (pretty much at the bottom) - contributors at Apache Beam are encouraged to do code reviews as well - in particular if the original patch comes from a committer, the idea is that such a review is enough for merging the code.
Maybe this is something for an InnerSource Pattern instead of the LearningPath though.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Lisez la section du parcours d’apprentissage Trusted Committer part 2 et l’article lié sur la création de communauté Apache Beam, en particulier sa discussion sur la revue du code par les contributeurs. Décidez si le contenu doit être mis à jour dans le parcours d’apprentissage ou consigné comme InnerSource Pattern ; la tâche sera terminée lorsque l’emplacement et la portée proposés auront été approuvés et que le contenu d’apprentissage concerné aura été révisé.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Domaine
- documentation
- Type d'issue
- Documentation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- À clarifier
- Accessibilité débutants
- 35/100