dzbaker
dzbaker
CCB 25 April 2024: Will review next week when @jphickey is back.
CCB 2 May 2024: Approved pending BVT run.
CCB 26 October 2023: Corwin will reach out to Kequan to identify which change specifically this is referring to.
> @skliper Do you mean _combine_ these (opposing) functions together, or separate out the common logic into _another_ function? I'm thinking the latter. @dmknutsen, what do you think?
Hi @thnkslprpt, apologies about the delay with this PR. I'd like to get it into the next round of PR reviews. Would you be able to resolve the merge conflict/unit...
@thnkslprpt I reverted the original (#383) so we could move it to the next release. Does everything look good here?
Thanks @thnkslprpt for making these fixes! Apologies about the very long delay in merging these changes; I'd like to get your event ID updates into the next round. Would you...
> Heads up @dmknutsen @dzbaker, looks like there's some real failures that started recently in CI: https://github.com/nasa/cFS/actions/runs/3230286177 (also causing the issue w/ this one). There's also quite a few (8...
> Looks like the org owners have to uninstall the lgtm.com app from the repos to actually remove the analysis and related warnings. I'll put in a request. Ok, thanks!
CCB 2 May 2024: Approved pending BVT run.