-
Notifications
You must be signed in to change notification settings - Fork 1.3k
@DaanHoogland
Description
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 #10975 (review)
Metadata
Metadata
Assignees
Type
Projects
Status
In Review