Drop coveralls code-coverage flow?

[rzo1]

Can we drop the coveralls github action?

I did a quick search at ASF level and I think, that only datasketches-cpp is using it. So might be good to just drop it and rely on other test coverage report metrics, which are more common at ASF level.

[lewismc]

SonarCloud.io is very good.

[sigee]

Hi,
For checking and requiring a minimum test coverage you could use jacoco as it is used in apache commons project.

There is no generated badge for readme, but the pipeline could work perfectly by require a minimum coverage for different metrics (line coverage, branch coverage, class coverage, etc.).
Jacoco is already part of this project, so only the configuration is needed here.

What do you think?

[rzo1]

I am using Jacoco in most of my projects ;-) - I have no objections in replacing it.