Hi Gary.
Small format/style request 1): Please add a space after the "&&" in this line: if (query_string.empty() &&scope->only_in_search() == true)
Style request 2): Please use four spaces instead of tabs in this project. For example indentation looks wrong on line 110 below.
Most new features need to be documented in the README.md. Do you think this does?
Please refresh the branch here and I will approve.
Thanks, Kyle
« Back to merge proposal
Hi Gary.
Small format/style request 1): string. empty() &&scope- >only_in_ search( ) == true)
Please add a space after the "&&" in this line:
if (query_
Style request 2):
Please use four spaces instead of tabs in this project. For example indentation looks wrong on line 110 below.
Most new features need to be documented in the README.md. Do you think this does?
Please refresh the branch here and I will approve.
Thanks,
Kyle