To allow the quick start of a new KDevelop plugin
Details
Details
- Reviewers
kfunk - Group Reviewers
KDevelop - Commits
- R32:6df9566854e5: Add an app_template for a simple KDevelop plugin
Diff Detail
Diff Detail
- Repository
- R32 KDevelop
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
Rest LGTM, great!
app_templates/cpp/CMake/cmake_kdevplugin/README | ||
---|---|---|
1 ↗ | (On Diff #9280) | Turn into a README.md (Markdown syntax) maybe? Way more common these days. |
23 ↗ | (On Diff #9280) | "... check the output of make install to see where the plugin is installed" maybe? |
app_templates/cpp/CMake/cmake_kdevplugin/src/%{APPNAMELC}.cpp | ||
7 | Style: Add newline before, remove after |
app_templates/cpp/CMake/cmake_kdevplugin/README | ||
---|---|---|
23 ↗ | (On Diff #9280) | This was written with building from KDevelop UI ;) So make install might be too console-centric? |
app_templates/cpp/CMake/cmake_kdevplugin/README | ||
---|---|---|
17 ↗ | (On Diff #9280) | It really should be "add", but as you can see by the update the double "on" also confused me on second read, so had changed to: "on OS with xx bit system add". Does that work better? |
35 ↗ | (On Diff #9280) | As you can see by my update I did while you were commenting on the initial version, I agree :) |