D6867: Add PowerShell highlighting

2017-07-24 Thread Volker Krause
vkrause closed this revision. vkrause added a comment. Pushed to master. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D6867 To: vkrause, dhaumann, vonreth Cc: #frameworks

D6867: Add PowerShell highlighting

2017-07-24 Thread Dominik Haumann
dhaumann accepted this revision. dhaumann added a comment. This revision is now accepted and ready to land. Looks good to me. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D6867 To: vkrause, dhaumann, vonreth Cc: #frameworks

D6867: Add PowerShell highlighting

2017-07-24 Thread Volker Krause
vkrause updated this revision to Diff 17103. vkrause added a comment. Right, license is due to the original file. REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6867?vs=17094&id=17103 REVISION DETAIL https://phabricator.kde.org/D6867 AFFECTE

D6867: Add PowerShell highlighting

2017-07-24 Thread Dominik Haumann
dhaumann added a comment. Ah, and we need spellChecking="0" almost everywhere except for HereString and String, and comments right? REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D6867 To: vkrause, dhaumann, vonreth Cc: #frameworks

D6867: Add PowerShell highlighting

2017-07-24 Thread Dominik Haumann
dhaumann requested changes to this revision. dhaumann added a comment. This revision now requires changes to proceed. Looks almost good, but I think another iteration would be nice. I assume we cannot relicense to MIT? INLINE COMMENTS > powershell.xml:5 > + version="1" > + kateversion=

D6867: Add PowerShell highlighting

2017-07-24 Thread Volker Krause
vkrause created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY Based on https://github.com/jgm/highlighting-kate/pull/80/ with some fixes and improvements. REPOSITORY R216 Syntax Highlighting REVISIO