Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade CI clang-format to > 16 #1311

Open
kidharb opened this issue Mar 14, 2024 · 0 comments
Open

Upgrade CI clang-format to > 16 #1311

kidharb opened this issue Mar 14, 2024 · 0 comments

Comments

@kidharb
Copy link
Sponsor Contributor

kidharb commented Mar 14, 2024

Not really a bug and not really a feature.
It is more of an enhancement to the CI test suite.

Currently, we use clang-format14 in a Docker container whenever a PR is updated.
This issue is to propose an upgrade to clang-format>16 such that we can

  1. Automatically check for newlines at the end of the file (instead of manually during a code review) InsertNewlineAtEOF: true
  2. Be able to handle newer C++ language instead of turning off clang for snippets like here
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Investigate
Development

No branches or pull requests

2 participants