gnodet-bot commented on code in PR #368: URL: https://github.com/apache/maven-clean-plugin/pull/368#discussion_r4324179634
########## src/site/markdown/examples/skipping-clean.md.vm: ########## @@ -26,7 +26,7 @@ under the License. # Skipping Clean Review Comment: The H1 heading and frontmatter `title` (line 2) still say "Skipping Clean", but `site.xml` was updated to "Skip Clean". Please align them: ```suggestion # Skip Clean ``` ########## src/site/markdown/index.md: ########## @@ -25,24 +25,26 @@ under the License. --> # Apache Maven Clean Plugin -The Clean Plugin is used when you want to remove files generated at build-time in a project's directory. + +The Clean Plugin removes the files that Maven generates in the directory of a project during a build. ## Goals Overview -The Clean Plugin only has one goal. +The Clean Plugin has one goal. + +- [clean:clean](./clean-mojo.html) removes the files that Maven generates in the working directory of a project. By default, it finds and deletes the directories in `project.build.directory`, `project.build.outputDirectory`, `project.build.testOutputDirectory`, and `project.reporting.outputDirectory`. -- [clean:clean](./clean-mojo.html) attempts to clean a project's working directory of the files that were generated at build-time. By default, it discovers and deletes the directories configured in `project.build.directory`, `project.build.outputDirectory`, `project.build.testOutputDirectory`, and `project.reporting.outputDirectory`. ## Usage General instructions for using the Clean Plugin can be found on the [usage page](./usage.html). Some more specific use cases are described in the examples given below. -If you still have questions about the plugin's usage, see the [FAQ](./faq.html) and feel free to contact the [user mailing list](./mailing-lists.html). The posts to the mailing list are archived and could already contain the answer to your question as part of an older thread. Hence, it is also worth browsing/searching the [mail archive](./mailing-lists.html). +If you have more questions about the plugin, see the [FAQ](./faq.html) and feel free to contact the [user mailing list](./mailing-lists.html). The posts to the mailing list are archived and could already contain the answer to your question as part of an older thread. Hence, it is also worth browsing/searching the [mail archive](./mailing-lists.html). Review Comment: This rewritten paragraph still contains several constructs that the rest of the PR removes: - "feel free to" (informal) - "could already contain" (hedging) - "Hence, it is also worth browsing/searching" (wordy) Suggested replacement: ```suggestion If you have more questions about the plugin, see the [FAQ](./faq.html) or contact the [user mailing list](./mailing-lists.html). The mailing list is archived — it may already have an answer to your question. You can also search the [mail archive](./mailing-lists.html). ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
