Skip to content

Conversation

@kou
Copy link
Member

@kou kou commented Aug 6, 2025

Rationale for this change

We want to add support for C++23.

What changes are included in this PR?

Add a CI job for C++23 as an extra job but its failure is ignored for now. Because we aren't C++23 ready for now.

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

@github-actions github-actions bot added the awaiting committer review Awaiting committer review label Aug 6, 2025
@raulcd raulcd added the CI: Extra Run extra CI label Aug 6, 2025
@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review labels Aug 6, 2025
@github-actions github-actions bot added awaiting change review Awaiting change review awaiting changes Awaiting changes and removed awaiting changes Awaiting changes awaiting change review Awaiting change review labels Aug 7, 2025
@kou kou merged commit 80addfa into apache:main Aug 8, 2025
9 of 10 checks passed
@kou kou removed the awaiting changes Awaiting changes label Aug 8, 2025
@kou kou deleted the ci-cpp-23 branch August 8, 2025 12:06
@conbench-apache-arrow
Copy link

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 80addfa.

There weren't enough matching historic benchmark results to make a call on whether there were regressions.

The full Conbench report has more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: Extra Run extra CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants