cloudstack icon indicating copy to clipboard operation
cloudstack copied to clipboard

Published 20 hours ago • verified publisher icon apache

the method `importVirtualMachineInternal(..)` now is 153 lines and takes 21 parameters.

Open DaanHoogland opened this issue 4 months ago • 0 comments

logic looks good, but the code here needs some refactorring I think (new PR?) the method now is 153 lines and takes 21 parameters. Not sure about cyclic complexity and I really don’t want to know anymore by now.

Originally posted by @DaanHoogland in https://github.com/apache/cloudstack/pull/10975#pullrequestreview-2931850389

DaanHoogland avatar Jun 16 '25 12:06 DaanHoogland