-
Notifications
You must be signed in to change notification settings - Fork 81
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
chore: move C++ format configuration #2554
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, CMakeLists formatting.
I'm going to try the suggested commit below, to see if it's narrow enough for the line length.
header-only/CMakeLists.txt
Outdated
# BSD 3-Clause License; see | ||
# https://github.com/scikit-hep/awkward-1.0/blob/main/LICENSE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't like the license being multi-line, even if it is a long line, but it looks like this is happening exactly once, in CMakeLists only (instead of every single C++ file). That's fine.
Actually, I'll try to see if it fits with 4 fewer characters.
# BSD 3-Clause License; see | |
# https://github.com/scikit-hep/awkward-1.0/blob/main/LICENSE | |
# BSD 3-Clause License; see https://github.com/scikit-hep/awkward/blob/main/LICENSE |
Taking your "it's fine" as tacit approval :) |
#2553 will also require a C++ release, so this PR can go in at the same time.