Skip to content

Refactor Merge Silo methods #538

@ybekdemir

Description

@ybekdemir

Merge silo and append silo methods currently has more than 300 lines of code. This is to much for two methods. We should separate this methods to smaller functions in order to increase readability and maintainability. Also, we should apply PEP8 to this methods and increase test coverage for them.

Methods to be refactor:

def merge_two_silos(mapping_data, lsid, rsid, msid)
def appendTwoSilos(mapping_data, lsid, rsid, msid):

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions