danhaywood commented on a change in pull request #590:
URL: https://github.com/apache/isis/pull/590#discussion_r648484147
##########
File path:
api/applib/src/main/java/org/apache/isis/applib/services/xactn/TransactionalProcessor.java
##########
@@ -25,7 +25,7 @@
import org.springframework.transaction.support.DefaultTransactionDefinition;
import org.apache.isis.commons.functional.Result;
-import org.apache.isis.commons.functional.ThrowingRunnable;
Review comment:
Domain apps depend on applib but as the dependency from applib -> common
is only scope="provided", in my view they shouldn't have a hard dependency on
common as well. So this was to ensure that everything that a domain app might
use is also in applib.
--
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
