This seems also OK

This one conflicts with D6645 on parser/test/lexertest.h and parser/test/lexertest.cpp.
Change either this one or D6645 so they can apply cleanly.
Thanks for your effort!

This one also looks good

Looks good to me

Looks good

Added test

Going to commit it tomorrow.
Objections?

In D5754#109711, @apol wrote:

In D5754#109708, @zhigalin wrote:

So, what about open!
?

True, my apologies, completely forgot. Where do you want the documentation exactly?

So, what about open!
?

Please add documentation to open! command

In D5122#107609, @geetamc wrote:

Added a comment explaining the need of "if(suCommandIsDigit)" block.

In D5122#107493, @kossebau wrote:

In D5122#107479, @zhigalin wrote:

• When you edit an UI file, make sure not to forget to increase it's version

This is a valid comment for kxmlgui rc files, but not for Qt ui files :) The version attribute of the <ui> rather specifies the version of the schema used, not of the content.

Hello and thanks for your effort.
Just a few considerations:

Ping

Thanks for your work, going to commit it

Rest LGTM

Fixed environmentProfileEntry

Good job :)

Agreeing with Kevin and Sven, splitting things is easier than putting parts together...

In D2220#42987, @kfunk wrote:

This has been pushed, right? (cf. https://git.reviewboard.kde.org/r/127979/)

See №127979
It should be OK now

In D429#8741, @mwolff wrote:

lgtm. Are you maybe interested in maintaining this plugin? Porting it to KDevelop 5 and having a release for that would be awesome, I think.