the method `importVirtualMachineInternal(..)` now is 153 lines and takes 21 parameters.
Offen
long-term-plan
no-issue-activity
type:technical-debt
- Vorherrschende Sprache
- Java
- Sterne
- 3.1k
- Forks
- 1.4k
- Ø Merge
- 6 T. 19 Std.
- Gemergte PRs (30 T.)
- 32
Beschreibung
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_
Beitragsleitfaden
Bewertung
Dieses Issue wurde noch nicht bewertet.