Closed dinhani closed 7 months ago
Thanks for the contribution!
This is definitely an improvement, one option that I also want to evaluate:
stdout
so that users can instead pipe it to whatever file they want to.I don't see a strong advantage of that option vs what you've made here (they are not even incompatible), so if you think that sending the file content to stdout
instead has some clear advantages we can do that. Otherwise I think what you've done here is great.
The main branch has deviated too much for shipping this PR, I've made a follow up that implements this feature https://github.com/gabotechs/dep-tree/pull/83, so it's safe to close this one.
Creates
render-path
flag, so the user can specify where the HTML file is generated instead of always using the hardcoded/tmp/index.html
.