https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26996

Joonas Kylmälä <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]
             Status|Needs Signoff               |Failed QA

--- Comment #2 from Joonas Kylmälä <[email protected]> ---
Thanks, this fixes the problem indeed. Few things to improve on the patch:

- In the commit title describe what the fix does, not what the problem is. E.g.
"Convert Elasticsearch indexer commit buffer size to int" and in the body
describe that it fixes the buffer counter not resetting and the reason why it
fixes it.

- In the commit title use "Bug XXXX: YYY" format which is used by every other
commit we have (now there is the "-" character instead).

- Add whitespace between the multiplication and parentheses.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to