We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
This section of code does not look properly indented, which makes it hard to read and reason about it:
This is because in my editor tabs are displayed as 4 spaces. We should not use tabs in the source code.
To fix:
The text was updated successfully, but these errors were encountered:
Agree.
We are using 4-spaces, no tabs. If you see anything else, feel free to correct.
I have never used clang-format. I would welcome a CI that demonstrates this in the CI.
Sorry, something went wrong.
I added .clang-fomat in this commit f32fb5d
.clang-fomat
And with those settings I ran it in the src directory on all source files, see this commit (ignoring whitespace changes)
Is this acceptable format/style?
I ran it locally on my laptop, I do not know how to integrate it in CI.
Yes, that looks great. Let's get your commits merged and I will put clang-format in the CI.
Thanks for this, it's really helpful. I'll suggest it for the netcdf-c repo.
edwardhartnett
Successfully merging a pull request may close this issue.
This section of code does not look properly indented, which makes it hard to read and reason about it:
This is because in my editor tabs are displayed as 4 spaces. We should not use tabs in the source code.
To fix:
The text was updated successfully, but these errors were encountered: