Cross-repo dependencies make maintenance PITA otherwise. Or maybe not cause cross-repo dependencies will be there in any case - needs discussion.
|integration/config||master||+1 -1||Zuul: Run all Kartographer tests alongside each patch to JsonConfig|
JCMapDataContent relies on both JsonConfig & Kartographer. But while it could still function properly without Kartographer, it cannot function at all without JsonConfig. This is similar to how MediaWiki api uses syntax highlighting - it makes the output nicer, but not required to exist.
18:44:15 Fatal error: Class undefined: Kartographer\SimpleStyleParser in /workspace/src/extensions/JsonConfig/includes/JCMapDataContent.php on line 48
Since moving JCMapDataContent from JsonConfig to Kartographer is not a good idea, would you suggest some other path to follow? To be honest I didn't get the time for further investigations on the code and I don't have an opinion yet.
I see it you had this concern before https://gerrit.wikimedia.org/r/c/integration/config/+/352279
I am going to bring this topic again for discussion and maybe ask for this patch to be merged.