apigear-io / cli

The ApiGear client as go project
https://apigear.io
MIT License
1 stars 2 forks source link

fix(qt_filters): qt_param filter now uses the prefix parameter #52

Closed dorotaphanSiili closed 1 year ago

w4bremer commented 1 year ago

In UE we prefix the name of the parameter, not the type. Here we prefix the type and not the name.

Should we have an option to prefix both or just have the prefix work differently depending on the technology?

dorotaphanSiili commented 1 year ago

🤔 we did not had any usecase (yet) where we need prefix parameter name, although it seems right to have it in similar manner. On the other hand we'd need two prefixes in all technolgies- of which one won't be used... All in all I am in favor of different technology == different filters Here in qt it is consistent with return filter which is adds prefix to type (as there is nothing else returned 😅 ) Also I wanted to check what does the cpp filter - but it also skips that parameter

jryannel commented 1 year ago

I will then later make a release and also get it into studio.