-
Notifications
You must be signed in to change notification settings - Fork 51
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
Adding dotfiles #91
Adding dotfiles #91
Conversation
I have added clang-format. @certik you should check, if the coding style suits your preference. I have done the formating with clang-format in a seperate commit so I can just revert it, if you don't agree with it. |
What do you think @certik? I have to revert the last commit anyway to fix the merge conflicts |
The formatting looks good, it's very similar to what we already used before. Let's merge the other PRs first, then we can merge this one. |
Yeah, I thought the same coding style is a good thing. Also I really like the chromium coding standards. |
96b3cee
to
56311ca
Compare
I have resolved everything! now we can merge this and proceed with #67. |
Thanks, that looks good. I squashed your commits to make the history a bit tidier. |
This pull-request ties together #76 and #30 by adding the default .gitignore provided by github here and all config files used by visual studio community, visual studio code, Jetbrains CLion and generated files by cmake as standalone.
clang-format will follow as well. The discussing is ongoing in #30.