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 gcov usage until it can be used. #15

Closed

Conversation

gberardi-pillar
Copy link

@gberardi-pillar gberardi-pillar commented Dec 13, 2023

Until tests can be run successfully (everest-framework depends on liblog), removing dependencies can help simplify our efforts at automating builds and tests.

To be clear, this is not a change to add CI/CD capability to this repo (presumably a different PR will come later). It is just to allow a repo that leverages this project to be able to successfully run and build its tests.

@gberardi-pillar
Copy link
Author

@shankari Can you review this PR?

CMakeLists.txt Outdated Show resolved Hide resolved
Signed-off-by: Gianfranco Berardi <[email protected]>
@gberardi-pillar
Copy link
Author

@shankari Sorry for the confusion about the other PR, but is it possible to get this PR looked at this morning? I haven't been able to make progress on updating everest-framework's test target name due to this PR being outstanding.

@hikinggrass
Copy link
Contributor

Picked up and fixed in #23

@hikinggrass hikinggrass closed this Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants