auto
auto copied to clipboard
Docs inconsistency for hook overview
Describe the bug
The diagram detailing when hooks (especially release lifecycle hooks) are invoked for each command are incredibly useful when writing a plugin or trying to understand how different plugins will behave together. However, when tracing through the code, I've noticed the following inconsistency(ies) between the diagram and the implementation:
-
next
flow invokesmakeRelease
hook, but is missing from diagram
This can lead to confusion especially in relation to existing discussions of potential hook refactoring, ie:
- https://github.com/intuit/auto/issues/1054#issuecomment-780236482
- https://github.com/intuit/auto/issues/917
Additional context
Since the diagrams are images, it makes it difficult for others to contribute when flows / hooks are changed. I'm not familiar with the static site generation tooling, but would it be possible to diagram within the markdown itself using something like https://mermaid-js.github.io/mermaid/#/flowchart?
Yeah that's would be a really good enhancement for the graph! I would def accept a PR for this!
@hipstersmoothie I would be glad to contribute on this. Can you please assign this to me?
I can start with migrating the existing diagrams to dynamic ones, and following to that, we can modify the graphs as needed. What do you say?
Just FYI for anyone visiting this thread, I am working on this. Thank you.