Remove coveralls

[purva-thakre]

Description

Fixes #220

The changes in this PR follow from https://docs.codecov.com/docs/github-2-getting-a-codecov-account-and-uploading-coverage#create-ci-pipeline

Todos

Notable points that this PR has either accomplished or will accomplish.

• TODO 1

Questions

• Question1

Status

• Ready to go

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2226c46) 97.2% compared to head (7f2a752) 97.2%.

Additional details and impacted files

@@          Coverage Diff           @@
##           master    #338   +/-   ##
======================================
  Coverage    97.2%   97.2%           
======================================
  Files         152     152           
  Lines        3046    3046           
  Branches      726     726           
======================================
  Hits         2961    2961           
  Misses         50      50           
  Partials       35      35           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[purva-thakre]

@vprusso marking this ready for a review. I don't really think I had to do a lot here considering codecov was already being used here. Coveralls was installed but not used by anything.

The existing workflow was pretty similar to what codecov needs https://github.com/codecov/codecov-action?tab=readme-ov-file#example-workflowyml-with-codecov-action

Are we even using coverage.xml ? I personally don't see the difference between only xml and specifying coverage.xml. The coverage was still uploaded as expected.

[vprusso]

🚀