search

fjuniorr / flowmapper

Mappings between elementary flows
MIT License
0 stars 1 forks source link

Reuse FlowProperty (name and synonyms) for other Flow properties (context, cas and unit) #58

Open fjuniorr opened 6 months ago

fjuniorr commented 6 months ago

There is a lot of duplication between FlowProperty used for Flow.name and Flow.synonyms and the classes:

It makes sense to reduce duplication probably through inheritance.

cmutel commented 6 months ago

I think this is nice to have - there are other things more important for now. I looked through the linked examples, the overlap is there but not huge.