search

Esri / arcgis-maps-sdk-toolkit-qt

ArcGIS Maps SDK for Qt Toolkit
Apache License 2.0
132 stars 61 forks source link

Guil8553/5362 remove qml #627

Closed GuillaumeBelz closed 4 months ago

GuillaumeBelz commented 4 months ago

Most of changes are just renaming.

I needed to do a lot of commits because there was a problem with validation script (fixed now)

GuillaumeBelz commented 4 months ago

I just saw I missed a lot of changes in documentations

GuillaumeBelz commented 4 months ago

For a reason I don't understand, I can't merge v.next to this branch to fix conflicts. Fixed.

GuillaumeBelz commented 4 months ago

Conflicts resolved

GuillaumeBelz commented 4 months ago

Looks good. I guess we might eventually want to rename toolkitcpp.pri to something else not including cpp but that doesn't need to be included in this work.

Yes, I thought about that. But there is toolkitwidgets project. I guess toolkit only is not explicit enough.

ldanzinger commented 4 months ago

changing the name of toolkitcpp.pri would likely have a far reaching impact, probably not worth it.