gitgabrio commented on code in PR #5765:
URL:
https://github.com/apache/incubator-kie-drools/pull/5765#discussion_r1517369878
##########
drools-model/drools-codegen-common/src/main/java/org/drools/codegen/common/AppPaths.java:
##########
@@ -47,6 +47,7 @@ public static AppPaths.BuildTool findBuildTool() {
private final boolean isJar;
private final BuildTool bt;
private final Path resourcesPath;
+ private final Path generatedResourcesPath;
Review Comment:
I look at the code. It would be possible to do as per your suggestion, but
IMO it would be unconsistent and more cumbersome.
About consistency:
`generatedResourcesPath` are semantically the same as `resourcesPath` ; and
both depends on the `bt` status: it make sense to manage them exactly in the
same way/same place
About cumbersomeness
I would need to introduce this check -> `if (bt == BuildTool.GRADLE) {`
inside the `getResourcesPAths` - but this check is already executed in the
constructor
So, even if it would be possible, I think it would make the overall result
worse: wdyt ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
