search

vaadin / vaadin-grid-flow

Vaadin Flow Java API for vaadin/vaadin-grid Web Component
https://vaadin.com/components/vaadin-grid
Other
14 stars 16 forks source link

chore: bump Web Component version to 5.7.0-alpha5 #1069

Closed DiegoCardoso closed 4 years ago

vaadin-bot commented 4 years ago

SonarQube analysis reported 32 issues

Top 10 extra issues

Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

  1. CRITICAL Grid.java#L2357: Remove usage of generic wildcard type. rule
  2. CRITICAL Grid.java#L3089: Define a constant instead of duplicating this literal "direction" 4 times. rule
  3. MAJOR Grid.java#: This file has 1,742 lines, which is greater than 750 authorized. Split it into smaller files. rule
  4. MAJOR Grid.java#L1893: Call "Optional#isPresent()" before accessing the value. rule
  5. MAJOR Grid.java#L1894: Either log or rethrow this exception. rule
  6. MAJOR Grid.java#L1913: Either log or rethrow this exception. rule
  7. MAJOR Grid.java#L2902: Call "Optional#isPresent()" before accessing the value. rule
  8. MAJOR Grid.java#L2912: Call "Optional#isPresent()" before accessing the value. rule
  9. MAJOR Grid.java#L3599: Rename "editor" which hides the field declared at line 1163. rule
  10. MAJOR Grid.java#L4011: Remove this unused method parameter "event". rule