<div dir="ltr"><div class="gmail_default" style="font-size:small">I'm the culprit for marking the PR as approved - sorry for the confusion.</div><div class="gmail_default" style="font-size:small">Luckily the PR still has Karstens 'do-not-merge' label. Once all conversations have been resolved or postponed, the label can be removed the PR will actually be merged. </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Dec 19, 2021 at 2:45 PM Jeremy Stanley <<a href="mailto:fungi@yuggoth.org">fungi@yuggoth.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2021-12-17 17:33:48 +0000 (+0000), Jeremy Stanley wrote:<br>
[...]<br>
> Since the PR got approved without addressing my review comment<br>
> (and those of others) about this, I'll re-raise the discussion<br>
> here instead.<br>
[...]<br>
<br>
Apologies for replying to myself, but discussion of these points has<br>
resumed in <a href="https://github.com/operate-first/community/pull/107" rel="noreferrer" target="_blank">https://github.com/operate-first/community/pull/107</a><br>
(post-approval).<br>
-- <br>
Jeremy Stanley<br>
_______________________________________________<br>
Openinfralabs mailing list<br>
<a href="mailto:Openinfralabs@lists.opendev.org" target="_blank">Openinfralabs@lists.opendev.org</a><br>
<a href="http://lists.opendev.org/cgi-bin/mailman/listinfo/openinfralabs" rel="noreferrer" target="_blank">http://lists.opendev.org/cgi-bin/mailman/listinfo/openinfralabs</a><br>
</blockquote></div>