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

IDE dev mode: add only reloadable local dependencies to the devmode context #13815

Merged

Conversation

aloubyansky
Copy link
Member

This PR is based on the investigations performed by @famod. Thanks a lot for your great efforts, @famod!

The IDE dev mode should be using only the re-loadable local dependencies of the app module instead of adding all the workspace modules as re-loadable.
The extension dependency filtering is extracted to a common utility class and applied to the IDE dev mode setup as well.
The logger in the DevMojo is now properly initialized.

@ghost ghost added area/core area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/maven labels Dec 10, 2020
@famod
Copy link
Member

famod commented Dec 10, 2020

There are some sysout calls in DependenciesFilter that should be removed, I guess.

Apart from that I confirm that this is (also) fixing this strange incremental/partial build error: #13740 (comment)
(originally found here: #13243 (comment))

Great job, our persistence has prevailed! 😁

PS: I didn't test dev mode with this change. Maybe you @glefloch want to check the Gradle side of things?

@glefloch
Copy link
Member

I can run some test tomorrow yes :)

@aloubyansky aloubyansky force-pushed the ide-devmode-only-reloadable-deps branch from 88cc785 to 3bd8763 Compare December 11, 2020 09:46
Copy link
Member

@glefloch glefloch left a comment

Choose a reason for hiding this comment

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

I just build the branch on my machine and tested it on a simple CLI app and a web app. Everything seems to work as expected :)

@aloubyansky aloubyansky force-pushed the ide-devmode-only-reloadable-deps branch from 3bd8763 to d6a820b Compare December 11, 2020 19:46
Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

OK, let's have this one in Beta1.

@gsmet gsmet merged commit af52e97 into quarkusio:master Dec 16, 2020
@ghost ghost added this to the 1.11 - master milestone Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/core area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/maven
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants