Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: expose Spring executor for Vaadin dev mode (#692) #694

Merged
merged 1 commit into from
Dec 4, 2020

Conversation

denis-anisimov
Copy link

Part of #691

Part of #691

# Conflicts:
#	vaadin-spring/src/test/java/com/vaadin/flow/spring/VaadinServletContextInitializerTest.java
@vaadin-bot
Copy link
Collaborator

SonarQube analysis reported 14 issues

  • CRITICAL 5 critical
  • MAJOR 3 major
  • MINOR 6 minor

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 VaadinServletContextInitializer.java#L183: Move constants to a class or enum. rule
  2. CRITICAL VaadinServletContextInitializer.java#L730: Refactor this method to reduce its Cognitive Complexity from 21 to the 15 allowed. rule
  3. CRITICAL VaadinServletContextInitializer.java#L744: Refactor this code to not nest more than 3 if/for/while/switch/try statements. rule
  4. CRITICAL VaadinServletContextInitializer.java#L751: Refactor this code to not nest more than 3 if/for/while/switch/try statements. rule
  5. CRITICAL VaadinServletContextInitializer.java#L757: Refactor this code to not nest more than 3 if/for/while/switch/try statements. rule
  6. MAJOR VaadinServletContextInitializer.java#L196: Define and throw a dedicated exception instead of using a generic one. rule
  7. MAJOR VaadinServletContextInitializer.java#L349: Define and throw a dedicated exception instead of using a generic one. rule
  8. MAJOR VaadinServletContextInitializer.java#L401: Define and throw a dedicated exception instead of using a generic one. rule
  9. MINOR VaadinServletContextInitializer.java#L643: Move this method into "RouteServletContextListener". rule
  10. MINOR VaadinServletContextInitializer.java#L651: Move this method into "WebComponentServletContextListener". rule

@denis-anisimov denis-anisimov merged commit 82dd927 into 15.0 Dec 4, 2020
@denis-anisimov denis-anisimov deleted the cp/15.0 branch December 4, 2020 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants