import-graph-visualizer icon indicating copy to clipboard operation
import-graph-visualizer copied to clipboard

Feature proposal - Reporters for JSON and HTML

Open hadrien-toma opened this issue 4 years ago • 2 comments

Description

I would like to add this young but already very nice tool on my CI and it would be awesome if a flag allows us to specify that we want a not live output, as JSON and HTML :yum:

Usage suggestion

JSON reporter

  • This JSON would be useful as a data source for other use cases (data processings) or visualizations
yarn run import-graph-visualizer \
    --no-browser-open \
    --report=json \
    --outputPath=dist/my-branch-name \
    --targets="apps/my-app/src/app/app.module.ts" \
    --sources=""

HTML reporter

  • As a code reviewer, I would like to quickly look at the difference introduced by the commit I am reviewing in term of imports in order to have the opportunity to visually identify an anomaly or a dissonance. For this use case, as the human eye is the added value, an HTML output is very nice.
yarn run import-graph-visualizer \
     --no-browser-open \
     --report=html \
     --outputPath=dist/my-branch-name \
     --targets="apps/my-app/src/app/app.module.ts" \
     --sources=""

Note

Thank you for developing and sharing import-graph-visualizer :pray: :heart:

hadrien-toma avatar May 29 '21 08:05 hadrien-toma

Would you be interested in serving a PR? 😁

BioPhoton avatar May 31 '21 11:05 BioPhoton

Hum it will depend on the amount of work to do but I would be pleased to! Maybe we can split this issue into different features?

Concerning the ability to save the served app, do you already have an idea of a clean implementation? This is the most important for me so I would like to implement this one prioritarly.

hadrien-toma avatar May 31 '21 16:05 hadrien-toma