-
Notifications
You must be signed in to change notification settings - Fork 123
document documentation guidelines #4486
Comments
Not sure if this appropriate here. But we should document all these guidelines within the repo. IMO it is not enough to link to some paper and an external page to explain decisions. There should at least be a short explanation within the repo, in case the external information goes away.
If you are refering to this I actually think that's a horribly useless way of documenting our libraries.
If we really want diagrams, they should probably defined with Mermaid. Then they can easily be updated by changing the code and are also rendered on GitHub. |
We probably should also have a |
I absolutely agree, will add!
I agree that this part of arc42 is not really ideal for FLOSS. I'll write something more specific to our situation, similar to plugins.
A few diagrams are very helpful for beginners.
They are not for you anymore, I also don't use them. They are useful, however, to get a first impression/overview what Elektra is composed of/what it can do, e.g., you see what lib depends on what or that Elektra has plenty of check and storage plugins.
It looks like it is doing the same as dot. But I agree that this is a much more modern.
It also has plenty of other integrations, e.g., GitLab, so it wouldn't be a lock-in situation. |
I think that's the idea. Basically dot, but works in the browser.
Also for the popular docs website frameworks. |
Yes, but I don't think the plugins diagram is very readable. I think it would be better to either reduce the number of plugins or show the same information in a different way, e.g. a table or some sort of color coding. For beginners the diagram should also show some information they need and can understand. Showing the dependencies between all the libraries before people even know what the libraries do might be overwhelming. |
document documentation guidelines now can be found here:
https://www.libelektra.org/devgettingstarted/documentation
The text was updated successfully, but these errors were encountered: