[GitHub] [maven] gnodet commented on a change in pull request #529: added an "ignore" option for checksumPolicy in documents

[GitBox <gi...@apache.org>]

gnodet commented on a change in pull request #529:
URL: https://github.com/apache/maven/pull/529#discussion_r712767193



##########
File path: maven-compat/src/main/mdo/profiles.mdo
##########
@@ -308,7 +308,7 @@ under the License.
           <name>checksumPolicy</name>
           <version>1.0.0</version>
           <description>What to do when verification of an artifact checksum fails - warn, fail, etc. Valid values are
-            "fail", "warn" (default) or "ignore"</description>
+            "fail" (default), "warn" or "ignore"</description>

Review comment:
       Given we list all the values, should we really keep the `- warn, fail, etc.` part after the `What to do when verification of an artifact checksum fails` sentence ?
   Maybe simply:
   `What to do when verification of an artifact checksum fails. Valid values are "fail" (default), "warn" or "ignore".`




-- 
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: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org