iurysza / module-graph

A Gradle Plugin for visualizing your project's structure, powered by mermaidjs
https://plugins.gradle.org/plugin/dev.iurysouza.modulegraph
MIT License
346 stars 13 forks source link

Add heading error with explanation on how to fix it #7

Closed StylianosGakis closed 1 year ago

StylianosGakis commented 1 year ago

๐Ÿš€ Description

For cases where the heading is not set properly, the plugin will give a detailed explanation of what goes wrong and how to fix it. Addresses #1

You may want to edit the text provided and/or even add the groovy syntax describing how to fix the issue if you wish. For the text itself I think this should suffice but definitely feel free to suggest a better error message if you wish.

๐Ÿ“„ Motivation and Context

See #1

๐Ÿงช How Has This Been Tested?

Inside module-graph/sample/build.gradle.kts, edit the moduleGraphConfig block and make a typo in the heading.set to be heading.set("### Dependency Diagrm") instead of heading.set("### Dependency Diagram"). Then run./gradlew :sample:createModuleGraph` Then you get a gradle error with the following message:

image

๐Ÿ“ฆ Types of changes

โœ… Checklist

StylianosGakis commented 1 year ago

Sorry for the multiple pushes, this is now ready for review. I was changing my mind about the error message, but this is a good start now I think, you may want to change it, if so feel free to either do it yourself or tell me about it and I can push an update to the PR

iurysza commented 1 year ago

Looks great! I think we can merge it like that. ๐ŸŽ‰