<table><tr><td style="">dhaumann added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D7175" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I see two issues that imo should be addressed:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Please add a test case in autotest/input/ or extend it if one already exists (did you run make test?)</li>
</ol>

<ol class="remarkup-list" start="2">
<li class="remarkup-list-item">How does it work, are we supposed to run this command from time to time manually? If so, then I'd be ok with that.</li>
</ol>

<ol class="remarkup-list" start="3">
<li class="remarkup-list-item">How high is the maintenance burden over time: I understand that this should this, but given we have a yaml file that needs to be maintained manually, I wonder whether we simply move the maintenance from one place to another and additionally introduce complexity (in terms of additional tooling) one first needs to understand before being able to fix things.</li>
</ol>

<ol class="remarkup-list" start="4">
<li class="remarkup-list-item">The kateversion is back to 2.4 and hard-coded colors are used again. Previously, it was set to 5.0, and used already the newly introduced default styles. I strongly suggest to keep the new default styles - we purposefully changed this some time ago, and I dislike that fact that we go a step back here.</li>
</ol></div></div><br /><div><strong>REPOSITORY</strong><div><div>R216 Syntax Highlighting</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7175" rel="noreferrer">https://phabricator.kde.org/D7175</a></div></div><br /><div><strong>To: </strong>turbov, dhaumann, Kate, Framework: Syntax Hightlighting, vkrause<br /><strong>Cc: </strong>cullmann, Frameworks<br /></div>