activemq-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (ARTEMIS-904) Remove cyclic dependencies from artemis-cli
Date Thu, 27 Apr 2017 15:42:04 GMT


ASF GitHub Bot commented on ARTEMIS-904:

Github user jbertram commented on the issue:
    Please squash the commits. Also, keep an eye on formatting. It looks like a few things
got changed in the last commit that seem incorrect.

> Remove cyclic dependencies from artemis-cli
> -------------------------------------------
>                 Key: ARTEMIS-904
>                 URL:
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>            Reporter: Bennet Schulz
> The project needs some smaller refactorings. Some methods should be moved to another
class which is responsible it (e.g. printBanner() in Artemis class should be moved to PrintData
> In addition to this simple case there are lots of other classes/packages which have cyclic
> I already did such a refactoring for artemis-jdbc. (ARTEMIS-830 Remove cyclic dependencies
from artemis-jdbc-store)
> We should track all our dependencies between classes and packages as the Spring guys
do since quite a while

This message was sent by Atlassian JIRA

View raw message