Skip to content

Conversation

@mahmoud-elmorabea
Copy link
Collaborator

This PR creates a script that enables us to generate Mermaid diagrams to better visualize the JSON events definitions. Those diagrams will be used in the documentation to make it easier for stakeholders to digest the log events definitions.

I've created a PR #4 to show case generated mermaid diagrams

@mahmoud-elmorabea mahmoud-elmorabea self-assigned this Apr 18, 2025
@mahmoud-elmorabea mahmoud-elmorabea marked this pull request as ready for review April 18, 2025 07:20
@mahmoud-elmorabea mahmoud-elmorabea requested a review from a team April 18, 2025 07:20
@mahmoud-elmorabea mahmoud-elmorabea marked this pull request as draft April 18, 2025 08:13
@mahmoud-elmorabea mahmoud-elmorabea force-pushed the MBL-1066-generate-mermaid-diagrams branch from 83f1d73 to 1d2ee0f Compare April 18, 2025 08:19
@mahmoud-elmorabea mahmoud-elmorabea marked this pull request as ready for review April 18, 2025 08:20
@mahmoud-elmorabea mahmoud-elmorabea force-pushed the MBL-1066-generate-mermaid-diagrams branch from 1d2ee0f to 8255ee9 Compare April 18, 2025 10:17
Copy link

@mrehan27 mrehan27 left a comment

Choose a reason for hiding this comment

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

Looks good

@mahmoud-elmorabea mahmoud-elmorabea changed the title Mbl 1066 generate mermaid diagrams Generate mermaid diagrams from JSON definitions Apr 18, 2025
@mahmoud-elmorabea mahmoud-elmorabea requested review from a team and Shahroz16 and removed request for Shahroz16 April 18, 2025 12:53
@mahmoud-elmorabea mahmoud-elmorabea changed the base branch from MBL-1065-automate-generating-json-logs-definition to main April 21, 2025 07:23
@mahmoud-elmorabea mahmoud-elmorabea force-pushed the MBL-1066-generate-mermaid-diagrams branch from 8255ee9 to e29a9b3 Compare April 21, 2025 07:27
@mahmoud-elmorabea mahmoud-elmorabea merged commit 5eef650 into main Apr 21, 2025
2 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.

3 participants