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: Don't revisit visited classes. #11447

Merged
merged 3 commits into from
Jul 21, 2021

Conversation

caalador
Copy link
Contributor

Do not revisit visited classes as
visited childer will not be rechecked
and possibly overwriten.

Always visit UI once as else we get
UI JsModule in fallback.

Fixes #9861

Do not revisit visited classes as
visited childer will not be rechecked
and possibly overwriten.

Always visit UI once as else we get
UI JsModule in fallback.

Fixes #9861
@caalador caalador added this to Iteration Reviews in OLD Vaadin Flow ongoing work (Vaadin 10+) via automation Jul 20, 2021
@caalador caalador force-pushed the issues/9861_dependency_scanning branch from 5c7bc96 to f946ead Compare July 20, 2021 11:35
Copy link
Contributor

@denis-anisimov denis-anisimov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Artur also mentions similar issue in the computeExporterEndpoints method.
This PR doesn't contain any changes related to this method and tests.

@caalador caalador force-pushed the issues/9861_dependency_scanning branch from 74bf5d1 to e9c03b9 Compare July 21, 2021 07:41
@vaadin-bot
Copy link
Collaborator

SonarQube analysis reported 1 issue

  1. CRITICAL FrontendDependencies.java#L374: Refactor this code to not nest more than 3 if/for/while/switch/try statements. rule

@denis-anisimov denis-anisimov merged commit f7c3bf8 into master Jul 21, 2021
OLD Vaadin Flow ongoing work (Vaadin 10+) automation moved this from Iteration Reviews to Done - pending release Jul 21, 2021
@denis-anisimov denis-anisimov deleted the issues/9861_dependency_scanning branch July 21, 2021 12:01
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with platform 22.0.0.alpha1 and is also targeting the upcoming stable 22.0.0 version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
OLD Vaadin Flow ongoing work (Vaadin ...
  
Done - pending release
Development

Successfully merging this pull request may close these issues.

FrontendDependencies loses dependencies in some cases
3 participants