Documentation about using the template from Git#40
Conversation
|
Thanks @nvdaes, this is a really solid improvement and the guide is now much clearer and more structured. The separation between setup, update, and troubleshooting makes it much easier to follow. I think we are very close to a consistent and usable workflow for non-Git experts, but there are a few key clarifications that would make the system fully deterministic and easier to reason about. 1. Global rule (important for consistency)I think we should explicitly define a single global priority rule for all files:
The three exceptions are:
Everything else (including .github/, .gitignore, workflows, etc.) should be considered as coming from AddonTemplate and used as the default version during updates. This rule would make conflict resolution much more predictable and avoid case-by-case decisions. 2. docs/ handling should be made more explicitThe current instructions are correct, but it might help to clearly state the intent:
Because of that, it should always be removed during the update process. For clarity, it might be better to present a single recommended approach: rather than presenting multiple equivalent alternatives, to reduce ambiguity for beginners. 3. Clarification on git restoreThe use of git restore is a very good improvement compared to older Git patterns. However, it might be worth clarifying its intent in this context:
One subtle point: when restoring from HEAD, it is effectively reinforcing the current add-on state, not pulling from the template, which may not be immediately obvious to beginners. Adding a short explanation could help avoid confusion. Overall suggestionI think the document is now very close to a solid “official workflow”, and these clarifications would mainly help:
Overall though, this is a big step forward compared to the previous copy/paste approach 👍 |
Description
Documentation about integration of the add-on template in add-ons using git.
Related issues
Fixes #37
Credits
This work comes from ideas provided by @abdel792, @CyrilleB79, and other contributors.