google / google/google-java-format

GJF adds <p> tag prior to the <div> tag

Offen
#1,180 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Java
Sterne
6.2k
Forks
937
Ø Merge
6 Min.
Gemergte PRs (30 T.)
3

Beschreibung

Detected at: https://github.com/checkstyle/checkstyle/issues/15865, User's got his answer at: https://github.com/checkstyle/checkstyle/issues/15865#issuecomment-2462352189

GJF adds `

` prior to `

` tag, but the style guide says that block-level tags should not be preceded by `

` tag:

https://google.github.io/styleguide/javaguide.html#s7.1.2-javadoc-paragraphs

```
HTML tags for other block-level elements, such as

    or , are not preceded with

    .
    ```

    Here's an example of it:

    ```
    01: /**
    02: * Plugin to standardize all directions to their appropriate abbreviation.
    03: *
    04: *

    Some javadoc here....

    05: *
    06: *

      07: *
    • Item - 1

    • 08: *
    • Item - 2

    • 09: *
    • Item - 3

    • 10: *

    11: *
    12: *

    This plugin assumes numbered streets have the correct suffix, when applicable.
    13: */
    14: public class Test {
    15: /** Some javadoc. */
    16: public static void main(String[] args) {
    17:
    18: }
    19: }
    20:
    ```

    ```
    $ java -jar .\checkstyle-10.20.1-all.jar -c .\google_checks.xml .\Test.java
    Starting audit...
    Audit done.
    ```

    ```
    $ java -jar .\google-java-format-1.24.0-all-deps.jar .\Test.java > TestUpdated.java
    ```

    ```
    01: /**
    02: * Plugin to standardize all directions to their appropriate abbreviation.
    03: *
    04: *

    Some javadoc here....

    05: *
    06: *

      07: *
    • Item - 1
      08: *
    • Item - 2
      09: *
    • Item - 3
      10: *

    11: *
    12: *

    This plugin assumes numbered streets have the correct suffix, when applicable.
    13: */
    14: public class Test {
    15: /** Some javadoc. */
    16: public static void main(String[] args) {}
    17: }
    18:
    ```

    ```
    java -jar .\checkstyle-10.20.1-all.jar -c .\google_checks.xml .\TestUpdated.java
    Starting audit...
    [WARN] C:\checkstyle testing\.\TestUpdated.java:4:4:

    tag should not precede HTML block-tag '

    ',

    tag should be removed. [JavadocParagraph]
    [WARN] C:\checkstyle testing\.\TestUpdated.java:14:1: The name of the outer type and the file do not match. [OuterTypeFilename]
    Audit done.
    ```

    Ignore the `OuterTypeFilename` violation.

    The formatter added `

    ` tag preceding to `

    ` tag, Checkstyle was complaining about it. But this did not happened in case of `
      ` tag, formatter didn't added preceding `

      ` tag to it.

      Edit: After formatting, GJF also removes closing li tags ( `` )

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.