Ken Matsui
Ken Matsui
@chinthakagodawita To respond to your question in https://github.com/chinthakagodawita/autoupdate/pull/282#pullrequestreview-874192982, If you are hard to keep the `dist` directory up-to-date, then you just update the `dist` directory when `autoupdate` is released. I...
@chinthakagodawita I would be grateful if you review this PR when you have enough time.
@chinthakagodawita Thank you for your response! I will take a look at the issue you've opened.
@an-tao > Thanks so much for the PR. It LGTM. Thank you for your review! > would u please add the execAsync and execCoro method to the FilterBuilder class? Sure!
It seems GitHub Actions on 092f40f571fe4e9ed14a3d4894413880fed20979 and e3e8c6973e6ee122d689e59fe3a5ef33248fa115 should be reran.
I do not have a compiler that implements coroutines, so the tests may fail.
Hi @an-tao, I've added tests for the builders. However, I am not sure they are working correctly because they are frequently skipped somehow. Let me ask one question about security....
Also, do you have any plan to drop the support of C++14? Using C++17 or over would clean up our codes. For example, `cpr` recently decided to set the minimum...
@an-tao > If you use placeholders for parameters, DbClient can prevent SQL injection. Thank you for your answer. Do you prefer to include the change to use placeholders in this...
@an-tao I've implemented the builder to use placeholders. Please let me know if you need additional implementation.