D16414: Changes to REXX syntax highlighting

Christoph Cullmann noreply at phabricator.kde.org
Thu Oct 25 19:41:45 BST 2018


cullmann requested changes to this revision.
cullmann added a comment.
This revision now requires changes to proceed.


  Hi, nice that you take care of fixing these issues.
  
  Two small things that could be improved:
  
  1. the version="2" needs to be bumped to 3
  2. there should be some unit test for the highlighting, you can put some test file into autotests/input (e.g. test.rex) this will make it much easier to keep this in a OK state in the future
  
  If you need more pointers on the testing, just ping us here, thanks!

REPOSITORY
  R216 Syntax Highlighting

REVISION DETAIL
  https://phabricator.kde.org/D16414

To: phily, #framework_syntax_highlighting, cullmann
Cc: cullmann, kwrite-devel, kde-frameworks-devel, michaelh, ngraham, bruns, demsking, sars, dhaumann
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181025/903ce299/attachment.html>


More information about the Kde-frameworks-devel mailing list