ibessonov commented on code in PR #5728:
URL: https://github.com/apache/ignite-3/pull/5728#discussion_r2073274977
##########
modules/configuration-api/src/main/java/org/apache/ignite/configuration/ConfigurationModule.java:
##########
@@ -142,20 +140,12 @@ default void
migrateDeprecatedConfigurations(SuperRootChange superRootChange) {
/**
* Returns a collection of prefixes, removed from configuration. Keys that
match any of the prefixes
* in this collection will be deleted.
- * <p>Use ignite.my.deleted.property for regular deleted properties</p>
- * <p>ignite.list.*.deletedProperty - for named list elements. Arbitrarily
nested named lists are supported</p>
+ * <p>Use {@code ignite.my.deleted.property} for regular deleted
properties</p>
+ * * <p>{@code ignite.list.*.deletedProperty} - for named list elements.
Arbitrarily nested named lists are supported</p>
Review Comment:
```suggestion
* <p>{@code ignite.list.*.deletedProperty} - for named list elements.
Arbitrarily nested named lists are supported</p>
```
##########
modules/configuration-api/src/main/java/org/apache/ignite/configuration/ConfigurationModule.java:
##########
@@ -142,20 +140,12 @@ default void
migrateDeprecatedConfigurations(SuperRootChange superRootChange) {
/**
* Returns a collection of prefixes, removed from configuration. Keys that
match any of the prefixes
* in this collection will be deleted.
- * <p>Use ignite.my.deleted.property for regular deleted properties</p>
- * <p>ignite.list.*.deletedProperty - for named list elements. Arbitrarily
nested named lists are supported</p>
+ * <p>Use {@code ignite.my.deleted.property} for regular deleted
properties</p>
+ * * <p>{@code ignite.list.*.deletedProperty} - for named list elements.
Arbitrarily nested named lists are supported</p>
Review Comment:
I think we don't use `</p>`, and that every `<p>` should only be present
after an explicit empty line. Does this javadoc pass our checks?
--
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]
