LGTM, thanks a lot!
Fri, Nov 17
I would miss the option to hide the build order child view. I almost never need it as even with many projects open (and I very often have that) most of the time I just need to build one project/target. Cross-project builds are almost never done.
So right now the build order view only adds visual noise and (more important) steals vertical space on small screens. So not happy with the second line of the patch.
Thu, Nov 16
Also fixes^Wworks-around https://bugs.kde.org/show_bug.cgi?id=358205 -- needs BUG: and FIXED-IN:
Not yet done is to hide the source formatting settings page from the application settings dialog in case no formatters are available.
@apol May I ask you to review 6 years old code of yours? :D -- why was ProjectChangesModel lazy-initialized?
;=) Actually I wanted that kdevelop just opened the project in the current working dir, if any if I start it without arguments ;=)
Code cleanup, rebase
@cullmann kdevelop -p DIR -- That's something you wanted to have, right? In case you ever wanted to look at KDevelop again :)
Add new feature:
kdevelop -p DIR
Nice idea indeed!
Wed, Nov 15
Thanks a lot for these patches!
Makes sense to me.
Tue, Nov 14
Just chiming in here, without having a final solution for your problem: This search path has been added for a reason -- to find a kioslave.exe next to the application binary in application bundles on macOS/Windows.
Mon, Nov 13
Sun, Nov 12
Mon, Nov 6
Could you just remove the problematic includes and try to compile without the dependency, IMHO I see no use of QtScript stuff.
Sat, Nov 4
Thu, Nov 2
Mon, Oct 30
Sat, Oct 28
@geetamc Note: Please fixup your user name to in Git (git config user.name).
Fri, Oct 27
Looks like it :)
LGTM (only had a relatively quick look)
Can't test this myself, so I'm putting this up for review.
For some reason arcanist replaces the author of the change...
Rest LGTM, very nice Gleb! You rock :)
Thu, Oct 26
Wed, Oct 25
Tue, Oct 24
Thanks a lot for the patch!
@mwolff Hmm, I think this feature could be useful at times. Being able to disable the Git plugin on a per-repo basis (and stored in Git's very own settings) sounds useful to me.
Mon, Oct 23
This definitely needs another revision. Please fix the outstanding issues.
Fri, Oct 20
i18n("Files to load, or directories to load as projects"), "[FILE[:line[:column]] | DIRECTORY]..."?
I could live with that.
Bonus points for adding two example usages at the bottom of the help text in a new Examples section(?)
What help text, the one printed by kdevelop --help?