Skip to content

Add documentation for plugin development and testing#89

Merged
velezd merged 2 commits into
rhinstaller:develfrom
velezd:devel
Apr 8, 2026
Merged

Add documentation for plugin development and testing#89
velezd merged 2 commits into
rhinstaller:develfrom
velezd:devel

Conversation

@velezd

@velezd velezd commented Apr 2, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@velezd velezd requested a review from PLukas2018 April 2, 2026 09:25
@velezd

velezd commented Apr 7, 2026

Copy link
Copy Markdown
Collaborator Author

extra-plugins test passed locally and I added commit that should hopefully fixed them in github.

PLukas2018
PLukas2018 previously approved these changes Apr 8, 2026

@PLukas2018 PLukas2018 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding the docs. I have just two nitpicks which I noticed, but I think it can be left as is and it LGTM. FYI the job seems still to be failing so I will leave it up to you if it is worth fixing 🤷‍♂️


As I am not too knowledgable about permian yet, I was trying to understand the concepts better -- I went over some plugins and this doc
https://github.com/rhinstaller/permian/blob/devel/doc/usage/usage.rst
was useful for me to better understand the concepts and how it works 🙂 .

Maybe it would make sense in the future also add some other doc describing overall architecture (maybe it already exists and I just did not find it). Some parts are already mentioned in
https://github.com/rhinstaller/permian/blob/devel/doc/usage/usage.rst
but maybe it would be worth adding some doc connecting individual terms -- command --> event --> GroupedWorkflow --> Workflow -> ...

Comment thread doc/development/howto-plugins.rst Outdated
Comment thread doc/development/howto-plugins.rst
@velezd velezd merged commit 14e550a into rhinstaller:devel Apr 8, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants