Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@haydenroche5
Copy link
Collaborator

I've not hooked this up to coveralls.io, yet. I want to get the coverage into a bit better shape before we start publishing the numbers. πŸ˜„

@haydenroche5 haydenroche5 self-assigned this Dec 13, 2022
@zfields
Copy link
Contributor

zfields commented Dec 14, 2022

I have always used gcovr (only because I stumbled across it first). However, I understand that lcov is more commonly supported.

I am fully supportive of moving forward with lcov. That being said, do you think it's worth doing some kind of comparative analysis (like 15 minutes of research), or are you already familiar with both and lcov is superior?

@haydenroche5
Copy link
Collaborator Author

I have always used gcovr (only because I stumbled across it first). However, I understand that lcov is more commonly supported.

I am fully supportive of moving forward with lcov. That being said, do you think it's worth doing some kind of comparative analysis (like 15 minutes of research), or are you already familiar with both and lcov is superior?

I actually initially wrote this to use gcovr rather than lcov. I switched to lcov because note-arduino is using it in run_all_tests.sh. I'm happy to use either one. I think it just comes down to which HTML we like more, as they seem to produce largely the same data. Here's an issue on the gcovr repo where one of the gcovr devs essentially says the same.

@zfields
Copy link
Contributor

zfields commented Dec 14, 2022

lol. Too good not to share with the whole world...
blues/note-arduino#67

I'm an idiot. lcov FTW.

Copy link
Contributor

@zfields zfields left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Side channel conversation

@haydenroche5 haydenroche5 merged commit 7ac062c into blues:master Dec 14, 2022
@haydenroche5 haydenroche5 deleted the coverage branch December 14, 2022 20:49
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.

2 participants