Skip to content

Add Github code coverage upload#13150

Open
Dreamsorcerer wants to merge 2 commits into
masterfrom
gh-code-cov
Open

Add Github code coverage upload#13150
Dreamsorcerer wants to merge 2 commits into
masterfrom
gh-code-cov

Conversation

@Dreamsorcerer

@Dreamsorcerer Dreamsorcerer commented Jul 15, 2026

Copy link
Copy Markdown
Member

Let's test this out for a bit and see how it compares to codecov.

@Dreamsorcerer Dreamsorcerer requested a review from webknjaz as a code owner July 15, 2026 15:04
@Dreamsorcerer Dreamsorcerer added bot:chronographer:skip This PR does not need to include a change note backport-3.14 Trigger automatic backporting to the 3.14 release branch by Patchback robot backport-3.15 Trigger automatic backporting to the 3.15 release branch by Patchback robot labels Jul 15, 2026
@github-code-quality

github-code-quality Bot commented Jul 15, 2026

Copy link
Copy Markdown

Code Coverage Overview

Languages: Python

Python / code-coverage/test

The overall coverage in the branch is 23%. Coverage data for the branch is not yet available.

Show a code coverage summary of the most covered files.
File 30ce9f8 +/-
web_ws.py 60%
client.py 58%
web_protocol.py 58%
client_reqrep.py 51%
web_urldispatcher.py 48%
connector.py 47%
helpers.py 46%
test_http_parser.py 19%
test_connector.py 11%
test_client_functional.py 9%

Updated July 15, 2026 15:27 UTC
Code Coverage is in Public Preview. Learn more and provide us with your feedback.

@codspeed-hq

codspeed-hq Bot commented Jul 15, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 83 untouched benchmarks
⏩ 83 skipped benchmarks1


Comparing gh-code-cov (30ce9f8) with master (cba121d)

Open in CodSpeed

Footnotes

  1. 83 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

Comment thread .github/workflows/ci-cd.yml Outdated
Comment thread .github/workflows/ci-cd.yml Outdated
Comment thread .github/workflows/ci-cd.yml Outdated
Co-authored-by: Sam Bull <aa6bs0@sambull.org>
@Dreamsorcerer

Copy link
Copy Markdown
Member Author

Results appear to be grouped by label, so we want all test coverage to use the same label, so results are merged together.

@codecov

codecov Bot commented Jul 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.97%. Comparing base (cba121d) to head (30ce9f8).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #13150   +/-   ##
=======================================
  Coverage   98.97%   98.97%           
=======================================
  Files         131      131           
  Lines       48824    48824           
  Branches     2546     2546           
=======================================
  Hits        48324    48324           
  Misses        376      376           
  Partials      124      124           
Flag Coverage Δ
Autobahn 22.12% <ø> (ø)
CI-GHA 98.90% <ø> (ø)
OS-Linux 98.67% <ø> (ø)
OS-Windows 97.02% <ø> (ø)
OS-macOS 97.93% <ø> (ø)
Py-3.10 98.12% <ø> (ø)
Py-3.11 98.37% <ø> (ø)
Py-3.12 98.46% <ø> (ø)
Py-3.13 98.45% <ø> (ø)
Py-3.14 98.46% <ø> (ø)
Py-3.14t 97.55% <ø> (-0.01%) ⬇️
Py-pypy-3.11 97.43% <ø> (-0.01%) ⬇️
VM-macos 97.93% <ø> (ø)
VM-ubuntu 98.67% <ø> (ø)
VM-windows 97.02% <ø> (ø)
cython-coverage 37.88% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@Dreamsorcerer

Copy link
Copy Markdown
Member Author

It's not merging per label, it's taking the last upload, making it pretty useless currently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-3.14 Trigger automatic backporting to the 3.14 release branch by Patchback robot backport-3.15 Trigger automatic backporting to the 3.15 release branch by Patchback robot bot:chronographer:skip This PR does not need to include a change note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant