Closed markrtucker closed 7 months ago
@AlexMapley / @t-margheim - I updated this with the latest changes from main
. Could you give this one more look before we mark it Ready for Review?
@colearendt - Are you able to help me understand which additional reviews are needed here? (We'd like to get this merged soon to avoid additional future merge conflicts.)
@markrtucker @t-margheim Who needs to approve what in here is still evolving and something we should still talk about more in a posit team meeting. I had one comment to be addressed, but I think this is good to merge after that. I reviewed and it all looks benign enough to merge with the reviews in here already.
Given Brian's approval, I am merging this PR.
Background
Last portion of https://github.com/rstudio/chronicle/issues/933. The chronicle team has made a new code release and we're updating our helm to match.
Summary