Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Remove include headers from the CMake sources #485

Closed
RiscadoA opened this issue Aug 31, 2023 · 0 comments · Fixed by #593
Closed

Remove include headers from the CMake sources #485

RiscadoA opened this issue Aug 31, 2023 · 0 comments · Fixed by #593
Assignees
Labels
A-Meta B-Build Related to the build configuration C-Code-Quality A section of code that is hard to understand or change D-Trivial Can be done in 30 minutes P-Backlog Nice to have, but not super important
Milestone

Comments

@RiscadoA
Copy link
Member

There's no reason to keep a list of all include files in the CMake build config, we should drop it.

@RiscadoA RiscadoA added C-Code-Quality A section of code that is hard to understand or change D-Trivial Can be done in 30 minutes S-Triage Issues whose priority still has to be figured out A-Meta B-Build Related to the build configuration labels Aug 31, 2023
@RiscadoA RiscadoA removed the S-Triage Issues whose priority still has to be figured out label Sep 12, 2023
@RiscadoA RiscadoA added this to the 0.0 Nursery milestone Sep 12, 2023
@RiscadoA RiscadoA self-assigned this Sep 16, 2023
@RiscadoA RiscadoA assigned luishfonseca and unassigned RiscadoA Sep 24, 2023
@RiscadoA RiscadoA added the P-Backlog Nice to have, but not super important label Sep 24, 2023
@luishfonseca luishfonseca linked a pull request Sep 26, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Meta B-Build Related to the build configuration C-Code-Quality A section of code that is hard to understand or change D-Trivial Can be done in 30 minutes P-Backlog Nice to have, but not super important
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants