7714056b8dde7a54a6a838cc7e727fe2dbd6ae44
@csomh @lholecek @gnaponie PTAL
rebased onto 71b7f3c599fb6b06fe6736861f4e0760d654b6d2
rebased onto b2cb5448dc6f460fcd21c0ecfbb02cb1eabc64de
s/A C3I client need/A C3I client needs/g
Is it worth explaining how and to who this list needs to be provided?
I've just put a couple of comments, beside those it looks great! +1
rebased onto 7714056b8dde7a54a6a838cc7e727fe2dbd6ae44
@gnaponie Thanks for the review.
I would say to "the C3I Team" in this document since. Maybe @mikeb at the moment?
@gnaponie Thanks for the review. Is it worth explaining how and to who this list needs to be provided? I would say to "the C3I Team" in this document since. Maybe @mikeb at the moment?
Should we eventually write that in the documentation? Or maybe that's not the right place?
@gnaponie Thanks for the review. Is it worth explaining how and to who this list needs to be provided? I would say to "the C3I Team" in this document since. Maybe @mikeb at the moment? Should we eventually write that in the documentation? Or maybe that's not the right place?
I would prefer to separate the internal process from the c3i-library. Ideally there should be an internal onboarding process and doc, I think
Yeah, it's a good point.
The PR is good to be merged IMHO.
@gnaponie Great! Thanks.
Let's also keep @mikeb known and see what his opinions will be.
Pull-Request has been merged by rayson
@csomh @lholecek @gnaponie PTAL