Changes almost all includes to be system includes. This tells the compiler to ignore warnings from them. Without the system flag, they get included as both system and normal includes. Cmake will pass these down as-is which is what caused the warning leakage. The only include that is not marked as system are the headers from this library, and its only enabled when not building as header-only. Since normal includes have priority over system includes, both can be set.
With these changes, you no longer have to limit how many warnings you want to enable, and works with CI automatically with no extra flags needed, so LIBREMIDI_CI is no longer needed, at least for warnings.
Changes almost all includes to be system includes. This tells the compiler to ignore warnings from them. Without the system flag, they get included as both system and normal includes. Cmake will pass these down as-is which is what caused the warning leakage. The only include that is not marked as system are the headers from this library, and its only enabled when not building as header-only. Since normal includes have priority over system includes, both can be set.
With these changes, you no longer have to limit how many warnings you want to enable, and works with CI automatically with no extra flags needed, so LIBREMIDI_CI is no longer needed, at least for warnings.