Files
Ranska d875330b73 clarification git flow dans CLAUDE.md et pr-conventions
PR toujours vers develop, master reserve aux releases taggees.
Ajout workflow detaille et section releases dans la doc.
2026-02-15 13:42:32 +01:00

2.2 KiB

Conventions pour les Pull Requests

Git Flow

Workflow obligatoire

Toutes les PR doivent cibler develop. Jamais master directement.

# 1. Partir de develop a jour
git checkout develop
git pull origin develop

# 2. Creer une branche feature
git checkout -b feature/<nom-descriptif>

# 3. Travailler, commiter
# ... commits ...

# 4. Pousser la branche
git push -u origin feature/<nom-descriptif>

# 5. Creer la PR vers develop
gh pr create --base develop --title "..."

# 6. Apres merge, nettoyer
git checkout develop
git pull origin develop
git branch -d feature/<nom-descriptif>

Releases

Les releases se font en mergeant develop → master et en taguant :

git checkout master
git merge develop
git tag v0.x.x
git push origin master --tags

Structure du message

Chaque PR doit contenir deux sections, une en francais et une en anglais, identifiees par les emojis drapeaux en en-tete :

## :fr: Resume

Description en francais...

## :gb: Summary

Description in english...

Contenu attendu

Titre

  • Court (< 70 caracteres)
  • En francais, descriptif de l'objectif principal
  • Exemples :
    • Fix imports nodes palette/ et text_size police
    • Ajout numerotation workflows et CI

Corps (chaque section FR et EN)

  • Quoi : lister les changements concrets
  • Pourquoi : expliquer le probleme resolu ou la fonctionnalite ajoutee
  • Comment : details techniques si pertinent (choix d'architecture, trade-offs)
  • Tests : comment verifier que ca marche

Checklist avant de soumettre

  • Tests mamba au vert (mamba spec/ --enable-coverage --format=documentation)
  • Pas de secrets ou fichiers temporaires dans le diff
  • Message de commit propre (pas de reference IA)
  • Branche feature creee depuis develop a jour
  • PR cible develop (pas master)

Exemple complet

## :fr: Resume

- Correction du chargement des nodes palette/ qui utilisaient importlib
  au lieu d'imports relatifs, causant des echecs isinstance()
- Ajout de PreviewImage dans tous les workflows de test

## :gb: Summary

- Fixed palette node loading that used importlib instead of relative
  imports, causing isinstance() failures at runtime
- Added PreviewImage nodes to all test workflows