Skip to content

Add file search

Rahul Vadhyar requested to merge pr/file-search into main

Created by: RahulVadhyar

It will only return file paths for now. However the results are also based on the file contents as indexed by baloo/krunner. Note that this is KDE specific.

Also, there is fix for CMake warning. No cmake warnings should come now.

Checklist(to mark the completed tasks, replace the space inside the brackets with an x):

  • Are the variable and function named correctly. Example: myNewVariable
  • Are the class and structs named correctly. Example: MyNewClass
  • Are the comments clear and concise.
  • Is the documentation updated.
  • There are no new warnings than before*.
  • Are the logs used correctly. Only logs are allowed. No print statements.

*The only reason new warnings may be allowed is if the code that produces the warnings are from third party code that we cannot modify.(Example: Qt, KDE Frameworks, git submodules etc.)

Merge request reports