search

vitruv-tools / Vitruv

View-based Development and Model Consistency Framework
http://vitruv.tools
Eclipse Public License 1.0
14 stars 19 forks source link

Make `EChange` generic over its content type #594

Closed JanWittler closed 1 year ago

JanWittler commented 1 year ago

Adapts to https://github.com/vitruv-tools/Vitruv-Change/issues/59

JanWittler commented 1 year ago

@vitruv-tools/maintainers Due to the new type safety for changes, it becomes apparent that the tool adapters are not working anymore. Besides probably many flaws there, right now they don't compile anymore because there is no functionality to transform from the recorded VitruviusChange<EObject> to the required VitruviusChange<Uuid>. This problem is not introduced by the PR but only uncovered.

I suggest to make the Tool Adapters legacy. I don't think it is worth it to hack some UUID resolver into the adapters to make it work. The proper way of fixing them would be migrate to the View-API but that would require rewriting most of the code anyways.