search

SOCI / soci

Official repository of the SOCI - The C++ Database Access Library
http://soci.sourceforge.net/
Boost Software License 1.0
1.41k stars 477 forks source link

Documentation: Fix typo #986

Closed Tectu closed 2 years ago

Tectu commented 2 years ago

Fix a minor typo in docs/types.md

vadz commented 2 years ago

Thanks for the fix! I will merge it soon.

Tectu commented 2 years ago

Closing != Merging :p

vadz commented 2 years ago

I prefer not to create merge commits for the single commit branches as they seem useless to me and GitHub doesn't provide any way to mark a PR as "merged" if it was cherry-picked from outside the web UI (which I'd rather not use if possible), so, unfortunately, we have to live with this side effect. I don't think it makes any difference in practice, does it?

Tectu commented 2 years ago

I don't think it makes any difference in practice, does it?

Nah, I was just wondering whether you maybe accidentally hit the "comment & close" button which might lead to this PR being forgotten about as it wouldn't show up as an open/active PR anymore :D