search

MaVoix / MaVoixSupportGenerator

Web form and tools to generate avatar, wallpapers and support overlays for social networks and others communication resources.
3 stars 2 forks source link

Removing configuration from repository (V2) #6

Closed oceanBigOne closed 7 years ago

oceanBigOne commented 7 years ago

Removing configuration from repository Adding tmp folder in gitignore Adding /.idea/ and OS folders in gitignore

ColinMaudry commented 7 years ago

Bon, la PR aurait dû venir d'un fork perso (le mien c'est ColinMaudry/MaVoixSupportGenerator) ou bien d'une branche interne à ce repo dédiée à la PR.

Pour cette fois, je vais implémenter les changements d' @oceanBigOne moi-même et faire la PR.

Les changements d' @oceanBigOne sont en place dans mavoix, et il a créé une PR pour les merger dans master. On va effectivement merger mavoix vers master un jour, mais pas maintenant.

Donc, pour fonctionner en mode PR plutôt que via des contributions non concertées, je recommande de procéder de la manière suivante, qui ne nécessite pas de créer des forks persos du repo :

  1. En local, vous créez une branche à partir de la branche de développement dev (elle n'existe pas encore, mais en fait ce sera mavoix renommée). La branche dev sera la branche qui sera la version de développement.
  2. Vous nommez cette branche d'un nom succinct qui décrit la fonctionnalité que vous ajouter (je vais l'appeler nvlle-fonctionnalite)
  3. Si vous souhaitez collaborer sur le développement de cette fonctionnalité, poussez la nouvelle branche vers le repo central (MaVoix/MaVoixSupportGenerator) pour que les autres collabos la pull
  4. Quand vous avez testé vos modifications dans nvlle-fonctionnalite, vous la poussez sur le repo central.
  5. Si la fonctionnalité est terminée, vous créez une PR de la branche nvlle-fonctionnalite vers dev au niveau central, dans l'interface de Github.com.
  6. Si vous pensez que quelqu'un en particulier devrait y jeter un oeil, vous le mettez en reviewer
  7. Si la fonctionnalité est acceptée, on accepte la PR, elle est donc mergée dans dev.
  8. La prochaine fois que l'on pull dev sur notre ordinateur, la nouvelle fonctionnalité est en place (la branche nvlle-fonctionnalite peut alors être supprimée).

Au prochain épisode : Comment on fait une release sur master ?