Skip to content
This repository has been archived by the owner on Aug 15, 2024. It is now read-only.

Point to the classes dir of processors that are other subprojects #113

Merged
merged 10 commits into from
Sep 2, 2019

Conversation

dansanduleac
Copy link
Contributor

@dansanduleac dansanduleac commented Aug 13, 2019

Before this PR

If a project defines a project with annotation processors (A) that another project (B) then consumes, in the same gradle build, it's necessary to keep doing ./gradlew idea every time you change the code in A, in order to see the new classes.

This is onerous.

After this PR

==COMMIT_MSG==
Configure inter-project annotation processor dependencies correctly in IDEA.
==COMMIT_MSG==

This PR is blocked on #115.

@dansanduleac dansanduleac marked this pull request as ready for review August 14, 2019 13:36
@dansanduleac dansanduleac requested a review from a team as a code owner August 14, 2019 13:36
@dansanduleac dansanduleac reopened this Aug 14, 2019
@felixdesouza
Copy link

ping on this please 🙏

def outputDir = dependencyModule.outputDir
?: project.rootProject.file("${projectOutputDir}/production/${dependencyModule.name}")
log.lifecycle("Configuring annotation dependency ${project.path} -> ${dependencyProject.path} with output dir: $outputDir")
return outputDir
Copy link
Contributor Author

Choose a reason for hiding this comment

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

intent here is to make the annotation-processor classpath entry point to intellij's output directory for that dependencyProject, so that changes to dependencyProject can be seamlessly picked up when compiling "this" project (variable project)

@bulldozer-bot bulldozer-bot bot merged commit 3c90e29 into master Sep 2, 2019
@bulldozer-bot bulldozer-bot bot deleted the ds/resolve-processor-subprojects-classes-dir branch September 2, 2019 10:48
@svc-autorelease
Copy link
Collaborator

Released 3.1.0

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants