Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add script to generate mermaid graph of automations #26432

Merged
merged 22 commits into from
Oct 27, 2023
Merged

Add script to generate mermaid graph of automations #26432

merged 22 commits into from
Oct 27, 2023

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Oct 24, 2023 •
edited by github-actions bot
Loading

Requires #26426
Fixes #26424

Example of what it generated: https://gist.github.com/t3chguy/db4750121ebabac5274a05e9f83701bb

It is not perfect, we might want to find something which lays out more responsively than Mermaid.

This change is marked as an internal change (Task), so will not be included in the changelog.

@t3chguy
Add mdbook-mermaid
778f634 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Add script to generate mermaid graph of automations
ae6f0ad 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Fix types
c74cd71 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Fix titles
53d79d7 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
DRY
859b2a5 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Improve comments
0fbbca6 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Improve debug
22382d6 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Fix Node IDs
01685db 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Fix on param filtering
8344595 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Iterate
b9fbdbf 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Fix graph components
a0c3da6 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Add label to workflow_run edges
8b4a525 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Add crontab parser
0e29ab8 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Fix ID edgecase
f30c741 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy t3chguy added the T-Task Tasks for the team like planning label Oct 24, 2023
@t3chguy
Tidy
225e870 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Add hack to bifurcate Manual trigger for better graphs
68ea876 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Fix matrix strategy
ec5a514 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
Iterate
df0a5c5 
Signed-off-by: Michael Telatynski <[email protected]>
Base automatically changed from t3chguy/mdbook-mermaid to develop October 26, 2023 07:40
@t3chguy
Merge branch 'develop' of https://github.com/vector-im/element-web in…
b82e49e 
…to t3chguy/mdbook-mermaid-workflows

# Conflicts:
#	yarn.lock
@t3chguy t3chguy marked this pull request as ready for review October 26, 2023 07:42
@t3chguy t3chguy requested review from a team as code owners October 26, 2023 07:42
@t3chguy
Improve naming for matrix job subgraph
7c4d3fb 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy
delint
adb2e82 
Signed-off-by: Michael Telatynski <[email protected]>
germain-gg
germain-gg approved these changes Oct 27, 2023
View reviewed changes
Copy link
Contributor

@germain-gg germain-gg left a comment

Choose a reason for hiding this comment

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

Looks fairly sane, but equally difficult to review in dept given the large PR

scripts/gen-workflow-mermaid.ts Outdated Show resolved Hide resolved
@t3chguy
Add some comments and improve types
6164754 
Signed-off-by: Michael Telatynski <[email protected]>
@t3chguy t3chguy merged commit e5d4d51 into develop Oct 27, 2023
19 checks passed
@t3chguy t3chguy deleted the t3chguy/mdbook-mermaid-workflows branch October 27, 2023 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@germain-gg germain-gg germain-gg approved these changes

@kerryarchibald kerryarchibald Awaiting requested review from kerryarchibald kerryarchibald is a code owner automatically assigned from element-hq/element-web-team

Assignees
No one assigned
Labels
T-Task Tasks for the team like planning
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Statically visualise our workflow hierarchy
2 participants
@t3chguy @germain-gg