Apply suggestions from code review

Co-authored-by: Sally MacFarlane <macfarla.github@gmail.com>
Signed-off-by: Fabio Di Fabio <fabio.difabio@consensys.net>

CHANGELOG.md

@@ -6,7 +6,7 @@
 
 - Add ABI-decoded revert reason to `eth_call` and `eth_estimateGas` responses [#5705](https://github.com/hyperledger/besu/issues/5705)
 - Layered transaction pool implementation is now stable and enabled by default, so the following changes to experimental options have been done [#5772](https://github.com/hyperledger/besu):
-  - `--Xlayered-tx-pool` is gone, to select the implementation use the new `--tx-pool` option with values `layered`(default) or `legacy`
+  - `--Xlayered-tx-pool` is gone, to select the implementation use the new `--tx-pool` option with values `layered` (default) or `legacy`
   - `--Xlayered-tx-pool-layer-max-capacity`, `--Xlayered-tx-pool-max-prioritized` and `--Xlayered-tx-pool-max-future-by-sender` just drop the `X` and keep the same behavior
 
 ### Additions and Improvements

besu/src/main/java/org/hyperledger/besu/cli/converter/DurationMillisConverter.java

@@ -20,7 +20,7 @@
 
 import picocli.CommandLine;
 
-/** The Percentage Cli type converter. */
+/** The Duration (milliseconds) Cli type converter. */
 public class DurationMillisConverter
     implements CommandLine.ITypeConverter<Duration>, TypeFormatter<Duration> {
 

besu/src/main/java/org/hyperledger/besu/cli/converter/exception/DurationConversionException.java

@@ -31,9 +31,9 @@ public DurationConversionException(final String value) {
   /**
    * Instantiates a new Duration conversion exception for negative value.
    *
-   * @param millis the value
+   * @param value the millis
    */
-  public DurationConversionException(final long millis) {
-    super(format("negative value '%d' is not allowed", millis));
+  public DurationConversionException(final long value) {
+    super(format("negative value '%d' is not allowed", value));
   }
 }