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

ci: switch to coveralls #127

Merged
merged 6 commits into from
Nov 27, 2021
Merged

ci: switch to coveralls #127

merged 6 commits into from
Nov 27, 2021

Conversation

onerandomusername
Copy link
Member

@onerandomusername onerandomusername commented Nov 24, 2021

Relevant Issues

closes #112

Description

remove codecov and switch to coveralls

@codecov
Copy link

codecov bot commented Nov 24, 2021

Codecov Report

Merging #127 (d0d7d78) into main (42e8674) will not change coverage.
The diff coverage is n/a.

❗ Current head d0d7d78 differs from pull request most recent head a0fb30a. Consider uploading reports for the commit a0fb30a to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##             main     #127   +/-   ##
=======================================
  Coverage   70.90%   70.90%           
=======================================
  Files          33       33           
  Lines        1619     1619           
  Branches      199      199           
=======================================
  Hits         1148     1148           
  Misses        439      439           
  Partials       32       32           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 42e8674...a0fb30a. Read the comment docs.

@onerandomusername onerandomusername force-pushed the coverage-artifact branch 2 times, most recently from 6cd3c86 to ac41566 Compare November 24, 2021 17:28
@onerandomusername onerandomusername changed the title ci: add coverage artifacts ci: switch to coveralls Nov 24, 2021
@coveralls
Copy link
Collaborator

coveralls commented Nov 24, 2021

Pull Request Test Coverage Report for Build 1509571551

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 20 unchanged lines in 4 files lost coverage.
  • Overall coverage remained the same at ?%

Files with Coverage Reduction New Missed Lines %
modmail/config.py 1 97.18%
tests/modmail/extensions/test_plugin_manager.py 4 81.82%
modmail/plugin_helpers.py 6 0%
modmail/utils/plugins.py 9 54.9%
Totals Coverage Status
Change from base Build #1: 0%
Covered Lines:
Relevant Lines: 0

💛 - Coveralls

@onerandomusername onerandomusername added skip changelog Skip the need for a changelog entry for a Pull Request a: CI Continuous Integration and Continuous Deployment a: tools Development related to our toolchain (Docker, poetry, flake8) p: normal Normal Priority s: needs review Ready for review and merge labels Nov 24, 2021
@onerandomusername onerandomusername force-pushed the coverage-artifact branch 7 times, most recently from b04799a to 244a9a8 Compare November 25, 2021 01:33
@onerandomusername onerandomusername merged commit eec59a3 into main Nov 27, 2021
@onerandomusername onerandomusername deleted the coverage-artifact branch November 27, 2021 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a: CI Continuous Integration and Continuous Deployment a: tools Development related to our toolchain (Docker, poetry, flake8) p: normal Normal Priority s: needs review Ready for review and merge skip changelog Skip the need for a changelog entry for a Pull Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Coverage: Migrate to coveralls?
3 participants