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

Add support for --graph=file.html #329

Merged
merged 8 commits into from
Dec 17, 2021
Merged

Add support for --graph=file.html #329

Changes from 2 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
54b69ca
Add support for `--graph=file.html`
styfle Dec 16, 2021
aff9f35
Merge branch 'main' into feat-add-html-graph-export
styfle Dec 16, 2021
13294d8
Fix formatting
jaredpalmer Dec 17, 2021
e58de98
Tweak html graph and automatically open in browser if tty
jaredpalmer Dec 17, 2021
d247a89
Merge branch 'main' into feat-add-html-graph-export
jaredpalmer Dec 17, 2021
42583bd
Remove println
styfle Dec 17, 2021
ecb8b99
Update docs
styfle Dec 17, 2021
ac2e0c7
Add missing html option
styfle Dec 17, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 25 additions & 2 deletions cli/internal/run/run.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -612,9 +612,32 @@ func (c *RunCommand) Run(args []string) int {
Verbose: true,
DrawCycles: true,
}))
hasDot := hasGraphViz()
ext := path.Ext(runOptions.dotGraph)[1:]
if ext == ".html" {
f, err := os.Create(runOptions.dotGraph)
if err != nil {
log.Fatal(err)
}
f.WriteString(`<!DOCTYPE html>
<html>
<head>
<meta charset="utf-8">
<title>Graph</title>
</head>
<body>
<script src="https://cdn.jsdelivr.net/npm/[email protected]/viz.js"></script>
<script src="https://cdn.jsdelivr.net/npm/[email protected]/full.render.js"></script>
<script>`)
f.WriteString("const s = " + graphString + ";new Viz().renderSVGElement(s).then(el => document.body.appendChild(el)).catch(e => console.error(e));")
jaredpalmer marked this conversation as resolved.
Show resolved Hide resolved
f.WriteString(`
</script>
</body>
</html>`)
f.Close()
}
hasDot := hasGraphViz()
if hasDot {
dotArgs := []string{"-T" + path.Ext(runOptions.dotGraph)[1:], "-o", runOptions.dotGraph}
dotArgs := []string{"-T" + ext, "-o", runOptions.dotGraph}
cmd := exec.Command("dot", dotArgs...)
cmd.Stdin = strings.NewReader(graphString)
if err := cmd.Run(); err != nil {
Expand Down