Skip to content
This repository was archived by the owner on Jul 16, 2025. It is now read-only.

Conversation

thomasrockhu-codecov
Copy link
Contributor

No description provided.

Copy link

codecov bot commented Feb 20, 2025

❌ 4 Tests Failed:

Tests completed Failed Passed Skipped
2860 4 2856 0
View the top 3 failed test(s) by shortest run time
api.temp.calculator.test_calculator::test_divide
Stack Traces | 0.001s run time
def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError
api.temp.calculator.test_calculator::test_divide
Stack Traces | 0.001s run time
def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError
api.temp.calculator.test_calculator::test_divide
Stack Traces | 0.001s run time
def
                test_divide():
                > assert Calculator.divide(1, 2) == 0.5
                E assert 1.0 == 0.5
                E + where 1.0 = <function Calculator.divide at 0x104c9eb90>(1, 2)
                E + where <function Calculator.divide at 0x104c9eb90> = Calculator.divide
                .../temp/calculator/test_calculator.py:30: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

Copy link

github-actions bot commented Feb 20, 2025

❌ 8 Tests Failed:

Tests completed Failed Passed Skipped
2852 8 2844 0
View the top 3 failed tests by shortest run time
tests.test_codecov_cli
Stack Traces | 0.000s run time
No failure message available
tests.commands.test_invoke_labelanalysis
Stack Traces | 0.000s run time
No failure message available
tests.commands.test_invoke_upload
Stack Traces | 0.000s run time
No failure message available

📣 Thoughts on this report? Let Codecov know! | Powered by Codecov

@@ -2,6 +2,7 @@
name: Build for PyPi

on:
pull_request:
Copy link
Contributor

Choose a reason for hiding this comment

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

I recommend calling this job from ci.yml instead and running builds for PRs unconditionally. Of course, you'll probably want to skip a portion of wheels in PRs (as an optimization), but could still build x86_64 — you're double-building them via pip install -e . anyway. But by integrating this instead, you could get closer to releasing exactly what you test.

@webknjaz
Copy link
Contributor

@thomasrockhu-codecov this probably needs to be rebased. The failing log says it installs test-results-parser v0.5.1:

  Installing collected packages: ijson, wrapt, urllib3, test-results-parser, sniffio, setuptools, regex, pyyaml, idna, h11, click, charset-normalizer, certifi, tree-sitter, sentry-sdk, requests, httpcore, anyio, responses, httpx, codecov-cli
  Successfully installed anyio-4.8.0 certifi-2024.12.14 charset-normalizer-3.4.1 click-8.1.8 codecov-cli-10.2.0 h11-0.14.0 httpcore-1.0.7 httpx-0.27.2 idna-3.10 ijson-3.3.0 pyyaml-6.0.2 regex-2024.11.6 requests-2.32.3 responses-0.21.0 sentry-sdk-2.20.0 setuptools-75.8.2 sniffio-1.3.1 test-results-parser-0.5.1 tree-sitter-0.20.4 urllib3-2.3.0 wrapt-1.17.2

However, Output was added in v0.5.2: codecov/test-results-parser#63.

main already pins it to v0.5.4 so if that's picked up, it'll probably address the issue.

@thomasrockhu-codecov thomasrockhu-codecov deleted the th/enable-py13-pypi-builds branch May 28, 2025 05:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants