search

SqliteModernCpp / sqlite_modern_cpp

The C++14 wrapper around sqlite library
MIT License
904 stars 156 forks source link

dev branch: cmake + catch + travis fix #145

Closed aminroosta closed 6 years ago

aminroosta commented 6 years ago

@zauguin please review.

zauguin commented 6 years ago

Is there a reason you want to add the .ycm_extra_conf.py? Especially the stdlib is very system specific, so I would not add a flag for this by default.

aminroosta commented 6 years ago

Thank you for reviewing :-) I've fixed the issues, we can merge then PRs like #126 or #125 into dev.

Is there a reason you want to add the .ycm_extra_conf.py

I removed the flags you mentioned, feel free to update this file if you are using YCM too. It can take a while for newcomers (who happen to be vim users) to set it up. I think there is no harm in keeping it.