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

Provide .clang-format & .clang-tidy #13

Open
marcluque opened this issue Oct 12, 2022 · 3 comments
Open

Provide .clang-format & .clang-tidy #13

marcluque opened this issue Oct 12, 2022 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@marcluque
Copy link
Contributor

This makes the coding style for the project consistent. Could be combined with a simple script or even local commit hook that runs it for all files.

@benzammour benzammour added enhancement New feature or request good first issue Good for newcomers labels Oct 12, 2022
@marcluque
Copy link
Contributor Author

I could make a proposal for a clang-format file. But I'd wait for #10 to be finished to not run into any struggles with merges.

@marcluque
Copy link
Contributor Author

Just a quick question for that. Should we use tabs or spaces for indentation? In the latter case, how many?

@marcluque
Copy link
Contributor Author

marcluque commented Oct 13, 2022

I've also noticed that the naming conventions are a bit inconsistent. Is there any specific rule that should be followed? I think that clang-tidy can enforce naming conventions. Just as an example camelCase vs snake_case for variables and struct members.

@marcluque marcluque changed the title Provide .clang-format Provide .clang-format & .clang-tidy Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants