Re: [PR] Add installation option with aqua to README [maven-mvnd]

["gnodet (via GitHub)" <gi...@apache.org>]

gnodet commented on code in PR #920:
URL: https://github.com/apache/maven-mvnd/pull/920#discussion_r3661587374


##########
README.adoc:
##########
@@ -75,6 +75,13 @@ Note: There are two formulae: `mvnd` installs the preview 2.x version (embedding
 sudo port install mvnd
 ----
 
+=== Install using https://aquaproj.github.io/[aqua]
+
+[source,shell]
+----
+$ aqua g -i apache/maven-mvnd

Review Comment:
   The `$` prompt prefix should be removed for consistency. Commit 31a212a5 ("chore: make install commands copy+pastable") deliberately removed `$` from all install commands because copying it causes `zsh: command not found: $` errors.
   
   Also, since `aqua init` is required before this command works, consider adding a brief note about the prerequisite:
   
   ```suggestion
   aqua g -i apache/maven-mvnd
   ```



-- 
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