Re: [PATCH v4 0/3] replay: make atomic ref updates the default
- From
Christian Couder <christian.couder@gmail.com>
- Date
- Oct 24, 2025, 09:39 UTC
- Message-ID
- <CAP8UFD0f030KHOJeM44k4wcxrEhFdzycvoaMtFZ-mTZ28LfJMw@mail.gmail.com>
- In-Reply-To
- <20251022185045.29256-1-siddharthasthana31@gmail.com>
On Wed, Oct 22, 2025 at 8:50 PM Siddharth Asthana <siddharthasthana31@gmail.com> wrote:
Show 5 quoted lines
> ## Testing > > All tests pass: > - t0450-txt-doc-vs-help.sh (now passing, was failing in v3) > - t3650-replay-basics.sh (all 18 tests pass)
Nit: a link to CI results on GitLab or GitHub might be better to show that all tests pass. Just saying "All tests pass" makes reviewers wonder if comprehensive CI tests on different OS/compilers/build systems were used or if you just tested on your machine.
Thanks.