-
Notifications
You must be signed in to change notification settings - Fork 69
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.
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
docs: Add Maven Plugin documentation #341
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #341 +/- ##
============================================
+ Coverage 91.31% 91.38% +0.06%
- Complexity 1218 1219 +1
============================================
Files 76 76
Lines 3168 3168
Branches 493 493
============================================
+ Hits 2893 2895 +2
+ Misses 164 163 -1
+ Partials 111 110 -1 ☔ View full report in Codecov by Sentry. |
@marcospereira wow, the quality of that maven plugin documentation is insane. Looks very good to me, I'll have a look at the other PR that blocks this merge. |
07a7ab7
to
2754ea2
Compare
2754ea2
to
a7455ca
Compare
Thanks! 😊 This is ready to go now. I will find the time to do the same for Gradle later. |
|
||
!!! warning "Clean all before precompiling" | ||
|
||
The `precompile` goal cleans the directory containing the compiled template classes every time it runs. In your local development environment, it may make more sense to use [hot reloading](hot-reloading.md). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The hint to hot reloading seems unrelated to the Clean all before precompiling
warning.
Co-authored-by: Niklas Keller <me@kelunik.com>
Requirements
This requires #339. I will keep it as a draft until the other PR is merged/closed.
What
Add a new documentation page to detail the parameters, goals, and usage of the Maven Plugin.