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

Merged
merged 2 commits into from
Dec 2, 2020

Conversation

denis-anisimov
Copy link

Part of #691

@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#L184: Move constants to a class or enum. rule
  2. CRITICAL VaadinServletContextInitializer.java#L792: Refactor this method to reduce its Cognitive Complexity from 21 to the 15 allowed. rule
  3. CRITICAL VaadinServletContextInitializer.java#L806: Refactor this code to not nest more than 3 if/for/while/switch/try statements. rule
  4. CRITICAL VaadinServletContextInitializer.java#L813: Refactor this code to not nest more than 3 if/for/while/switch/try statements. rule
  5. CRITICAL VaadinServletContextInitializer.java#L819: Refactor this code to not nest more than 3 if/for/while/switch/try statements. rule
  6. MAJOR VaadinServletContextInitializer.java#L197: Define and throw a dedicated exception instead of using a generic one. rule
  7. MAJOR VaadinServletContextInitializer.java#L399: Define and throw a dedicated exception instead of using a generic one. rule
  8. MAJOR VaadinServletContextInitializer.java#L452: Define and throw a dedicated exception instead of using a generic one. rule
  9. MINOR VaadinServletContextInitializer.java#L690: Move this method into "RouteServletContextListener". rule
  10. MINOR VaadinServletContextInitializer.java#L698: Move this method into "WebComponentServletContextListener". rule

@denis-anisimov denis-anisimov merged commit 641defe into master Dec 2, 2020
@denis-anisimov denis-anisimov deleted the 691-stop-frontend-tools branch December 2, 2020 10:56
denis-anisimov pushed a commit that referenced this pull request Dec 3, 2020
Part of #691

# Conflicts:
#	vaadin-spring/src/test/java/com/vaadin/flow/spring/VaadinServletContextInitializerTest.java
denis-anisimov pushed a commit that referenced this pull request Dec 3, 2020
Part of #691

# Conflicts:
#	vaadin-spring/src/test/java/com/vaadin/flow/spring/VaadinServletContextInitializerTest.java
denis-anisimov pushed a commit that referenced this pull request Dec 3, 2020
Part of #691

# Conflicts:
#	vaadin-spring/src/test/java/com/vaadin/flow/spring/VaadinServletContextInitializerTest.java
denis-anisimov pushed a commit that referenced this pull request Dec 4, 2020
Part of #691

# Conflicts:
#	vaadin-spring/src/test/java/com/vaadin/flow/spring/VaadinServletContextInitializerTest.java
denis-anisimov pushed a commit that referenced this pull request Dec 4, 2020
Part of #691

# Conflicts:
#	vaadin-spring/src/test/java/com/vaadin/flow/spring/VaadinServletContextInitializerTest.java
caalador pushed a commit that referenced this pull request Dec 14, 2020
Part of #691

# Conflicts:
#	vaadin-spring/src/test/java/com/vaadin/flow/spring/VaadinServletContextInitializerTest.java
manolo pushed a commit to vaadin/flow that referenced this pull request Feb 8, 2022
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