Pull Request sur GitHub pour contribuer : Bonnes pratiques?

Salut la communauté,

J’ai envie de participer à mon niveau en contribuant à améliorer la doc de Jeedom (corrections, mises à jour…).

Je suis plutôt nouveau (moins d’un an d’utilisation quotidienne) et en tant que nouvel utilisateur, il y a des précisions utiles que j’aurai aimé voir à l’époque et qui pourraient aider les autres nouveaux(elles) utilisateurs(rices).

Évidemment, sous le contrôle de la Jeedom Team qui donne son aval sur un éventuel « Merge » :wink: C’est la règle et c’est bien normal !

J’ai créé mon premier Pull request sur GitHub pour essayer, n’y connaissant rien (mais m’étant documenté), j’espère tout de même ne pas avoir fait de bêtise.

C’est ici:

Est-ce qu’une bonne âme pourrait me confirmer que c’est bon dans le process ? :slight_smile: :pray:

J’ai vu des propositions d’autres utilisateurs dans le répertoire (dont @noodom), parfois assez anciennes (25jours), du coup je voulais savoir si la team avait un rythme défini pour l’intégration ou le refus, par exemple 1 fois par mois ou par trimestre ?

Merci pour vos réponses :wink:
Bonne journée

1 « J'aime »

Merci, je viens de merger ton PR et les deux autres qui attendaient :wink:

2 « J'aime »

Yeah ! :+1:
Merci @kiboost, content d’avoir pu aider :wink:

Donc, cela se fait un peu au fil de l’eau lorsque vous avez le temps pour les analyse de PR ?
Pas évident de jongler j’imagine.

Ça me démange vraiment pour la suite avec d’autres PR, j’ai plein d’idées d’endroits à améliorer.
Mais je ne voudrais surtout pas vous « charger » d’avantage.

Donc pour la suite, on reste pareil, je propose et toi ou d’autres validez à l’avancement ?

Merci :clap:

Bonsoir,

Je veux bien savoir ou réaliser les modifications pour les scénarios par exemple, j’ai fait plusieurs tentatives ou on me répond « en anglais » que je ne le fait pas au bon emplacement.
Or, c’est ce qu’indique la documentation.

Merci.

Je jèterai un œil si tu t’y met.
Vas y doucement au début, si y’a des trucs qui vont pas on pourra en parler.

Je t’invite à bien lire ceci avant

https://doc.jeedom.com/fr_FR/contribute/doc

:beers:

La doc scénario, l’aide du core donc, c’est sur le repo du core pas de la doc.

D’ailleurs en 4.2 va falloir reprendre le changelog pour adapter la doc. A voir quand elle passera en bêta.

Et pourtant, je trouve la documentation des scénarios ici :

Et donc, ici aussi :

Il faut l’avouer, c’est pas simple pour s’y retrouver, dans ce cas, pourquoi est-elle présente à différents emplacements.

C’est parfait ! Merci.

J’avais effectivement parcouru le lien, et d’autres pour le familiariser avec GitHub qui est un tout autre et très vaste sujet :cat2:

Promis j’irai doucement…
Hâte de pouvoir aider :wink:

Dans le lien de @kiboost il est indiqué de toujours « bosser sur la version alpha ».
Charge ensuite à la team de merger en « master »

C’est ce qui est expliqué, la doc du core et des plugins officiels sont rapatriés automatiquement sur le git de la doc. Donc si tu fait un commit sur la doc du core sur le git de la doc il sera écrasé plus tard. Il faut le faire à la source.

1 « J'aime »

Ce sujet a été automatiquement fermé après 24 heures suivant le dernier commentaire. Aucune réponse n’est permise dorénavant.