No problem, Mihai, it was just a small mistake, easy to fix :)
You're consistently our top reviewer; without your work, things would move
much more slowly. Thanks for your incredible help with the review process!

Best,
Ruben



On Mon, Nov 9, 2026 at 4:26 PM Mihai Budiu <[email protected]> wrote:

> Sorry about this, I should have been more careful.
>
> Mihai
> ________________________________
> From: Ruben Q L <[email protected]>
> Sent: Monday, November 9, 2026 1:03 AM
> To: [email protected] <[email protected]>
> Subject: Re: CI failures caused by commit message validation on main
>
> Hi,
>
> It seems the merge of PR#5398 [1] (which was merged without squashing)
> broke the LintTest [2].
> Typically for this type of commit we use the "Site: ..." prefix, e.g. [3]
>
> How should we address this? Shall we go for an exceptional force-push right
> away?
>
> Best,
> Ruben
>
> [1] https://github.com/apache/calcite/pull/5398
> [2]
>
> https://ci-builds.apache.org/job/Calcite/job/Calcite-snapshots/2219/testReport/junit/org.apache.calcite.test/LintTest/testLintLog__/
> [3]
>
> https://github.com/apache/calcite/commit/9e8177360b983aabe9cd8bdfe3edc165451ddbb8
>
>
>
> On Mon, Nov 9, 2026 at 8:39 AM Vladislav Pyatkov <[email protected]>
> wrote:
>
> > Hi all,
> >
> > PR check filed due to two commits:
> >
> >    - docs: describe Wordfully's Calcite use
> >    - docs: add Wordfully to powered-by page
> >
> > Both trigger “Message must start with upper-case letter”.
> >
> > --
> > Vladislav Pyatkov
> >
>

Reply via email to