Initial clang-tidy configuration
This adds some configuration to clang tidy, more to come in the future as well as more fixes to the code to go along with it.
Thanks.
Thanks for your pull request! It looks like this may be your first contribution to cppfront. I've emailed you the Contributor License Agreement (CLA), and once it's signed I can look at your pull request. Thanks again for your contribution.
Signed.
Got it, thanks Radu!
Looks good, we just have the reflect.h merge conflict... usually the line numbers changes don't cause a conflict, and maybe the simplest workaround is to move the reflect.h/h2 changes to a separate PR?
Looks good, we just have the reflect.h merge conflict... usually the line numbers changes don't cause a conflict, and maybe the simplest workaround is to move the reflect.h/h2 changes to a separate PR?
I hid the initialization thing for now cause some didn't like the immediately invoked lambda expression..., without that the line number changes are no longer an issue
I think I'll leave it until you relax the rules for out parameters.
how can I get those workflows approved?
I'm sure Herb is busy with CppCon right now. The conference opens tomorrow and he has the opening keynote on Monday morning.
Thanks!