BHoM_Adapter: general refactoring-COLLECTION-recap

[alelom]

Collection of issues related to BHoM_Adapter general refactoring.

Issues

1. Non breaking changes & no expected impact on behaviour of Toolkits

• BHoM_Adapter: refactoring-Push-shallow to deep clone #86
• BHoM_Adapter: refactoring-ReplaceThroughAPI-simplification possible #89
• File_Adapter: minor refactor (extension if needed, param name) #111

2. Breaking changes limited to non-toolkits

• BHoM_Adapter: refactoring-PullTo-rename to "Move"  #91
• BHoM_Adapter: refactoring-UpdateProperty (Public method)- reconsider usage #109

3. Breaking changes

• #78
• BHoM_Adapter: refactoring-UpdateProperty (CRUD)-make private and limit to update tags #87
• BHoM_Adapter: refactoring-Pull-Should not have ReadResults() logic in low-level code #90
• BHoM_Adapter: refactoring-refactor the ReplaceInMemory #106
• File_Adapter: refactoring-refactor to match ReplaceInMemory refactoring #107
• BHoM_Adapter: refactoring-change the config parameter #108
• BHoM_Adapter: refactoring-remove ErrorLog in light of the Global error system #113
• BHoM_Adapter: refactoring-remove NextId and AssignId #115

Extra/to discuss

• BHoM_Adapter: refactoring-PullTo-implement batches #92
• BHoM_Adapter: consider whether to add an "info" output from the Push/Pull/etc #95
• BHoM_Adapter: refactoring- should all IBHoMObjects implement a default "base" comparer that when checking for equality first of all looks at the GUID? #99
• BHoM_Adapter-#116-refactoring-code structure, folders and files tidy up #117