If I could get a review on <https://github.com/osq...
# code-review
g
If I could get a review on https://github.com/osquery/osquery/pull/5981 I'd be grateful -- Teddy has already very kindly taken the time to go over it a couple times and I've addressed the comments (and gotten it back to passing tests)
t
it looks good to go mostly, I can make some changes to the CMake code (it looks like the indents at 4 when they are typically 2.
s
Seems okay to me. I had one question, but I’m pretty sure the answer is yes. I’ll thumb it, and leave it for Teddy to merge if he’s happy
g
oh, that's a good catch, the ordering is now predictable
s
Sweet! Thanks for fixing that