Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

CMake Scoped Compiler Flags #24

Merged
merged 10 commits into from
Jun 27, 2023
Merged

CMake Scoped Compiler Flags #24

merged 10 commits into from
Jun 27, 2023

Conversation

threeal
Copy link
Owner

@threeal threeal commented Jun 27, 2023

  • Removed global setting of CMAKE_CXX_STANDARD.
  • Replaced global setting of CMAKE_CXX_FLAGS with target_compile_options to specify warning flags.
  • Set warnings to be treated as errors by default.
  • Modified warning flags to be enabled only if the project is not included as a dependency.
  • Replaced global setting of CMAKE_CXX_FLAGS with target_compile_options and target_link_options to specify coverage flags.
  • Enhanced CMakeLists.txt with descriptive comments.

@threeal threeal added the feat label Jun 27, 2023
@threeal threeal self-assigned this Jun 27, 2023
@threeal threeal force-pushed the cmake-scoped-flags branch from d7fb17b to e2bde1f Compare June 27, 2023 14:44
@threeal threeal marked this pull request as ready for review June 27, 2023 15:28
@threeal threeal merged commit f50eaf3 into main Jun 27, 2023
@threeal threeal deleted the cmake-scoped-flags branch June 27, 2023 15:32
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant