Removed unnecessary @Local annotations and their respective imports#853
Conversation
|
cloudstack-pull-rats #657 SUCCESS |
|
cloudstack-pull-analysis #606 FAILURE |
|
cloudstack-pull-rats #661 SUCCESS |
|
cloudstack-pull-analysis #610 FAILURE |
|
Hi @critofolini, I think that there was a problem in your PR in class: SecurityGroupManagerImpl |
|
cloudstack-pull-rats #662 SUCCESS |
|
cloudstack-pull-analysis #611 FAILURE |
|
cloudstack-pull-rats #664 SUCCESS |
|
cloudstack-pull-analysis #613 FAILURE |
|
cloudstack-pull-rats #665 SUCCESS |
|
cloudstack-pull-analysis #614 SUCCESS |
|
@cristofolini, could you squash your commits into a single one? |
cristofolini
force-pushed
the
master-lrg-cs-hackday-006
branch
from
9e4e17d to
b4c8db4
Compare
|
cloudstack-pull-rats #666 SUCCESS |
|
@rafaelweingartner Done. |
|
cloudstack-pull-analysis #615 SUCCESS |
rafaelweingartner
force-pushed
the
master-lrg-cs-hackday-006
branch
from
b4c8db4 to
a5ca762
Compare
…-006 Removed unnecessary @Local annotations and their respective importsFollowing @rafaelweingartner 's findings in PR #714 that many spring beans contained an @Local annotation, we've decided to remove said annotations and their imports from the ComponentLifecycleBase class and its subclasses seeking a reduction of a few hundred lines of useless code. I had already opened a pull request for this (#853) but at some point my commit disappeared from the PR entirely, showing no new changes in code, which caused it to be merged automatically (with no changes). * pr/1102: Removed unnecessary @Local annotations and their respective imports from the ComponentLifecycleBase class and its subclasses. Signed-off-by: Remi Bergsma <github@remi.nl>
4 participants
Following @rafaelweingartner 's findings in PR #714 that many spring beans contained an @Local annotation, we've decided to remove said annotations and their imports seeking a reduction of a few hundred lines of useless code.