Hi,
I noticed that the repository currently contains two architecture diagrams in the README:
One appears under the Table of Contents section
https://github.com/DevStrikerTech/datahelm?tab=readme-ov-file#table-of-contents
Another appears under High Level Architecture
https://github.com/DevStrikerTech/datahelm#high-level-architecture
Since the architecture diagram logically belongs to the High Level Architecture section, it may be better to keep the diagram only there rather than placing one near the Table of Contents.
Additionally, if the diagram image background could be made transparent, it would likely look cleaner and integrate better with GitHub’s light and dark themes.
Suggested improvements
Move/remove the architecture diagram under Table of Contents
Keep the architecture diagram under High Level Architecture
Use a transparent background image for better visual appearance
Thanks!
Hi,
I noticed that the repository currently contains two architecture diagrams in the README:
One appears under the Table of Contents section
https://github.com/DevStrikerTech/datahelm?tab=readme-ov-file#table-of-contents
Another appears under High Level Architecture
https://github.com/DevStrikerTech/datahelm#high-level-architecture
Since the architecture diagram logically belongs to the High Level Architecture section, it may be better to keep the diagram only there rather than placing one near the Table of Contents.
Additionally, if the diagram image background could be made transparent, it would likely look cleaner and integrate better with GitHub’s light and dark themes.
Suggested improvements
Move/remove the architecture diagram under Table of Contents
Keep the architecture diagram under High Level Architecture
Use a transparent background image for better visual appearance
Thanks!