Skip to content

fix: CI being broken#17

Merged
spalmurray merged 2 commits intomainfrom
what
May 14, 2025
Merged

fix: CI being broken#17
spalmurray merged 2 commits intomainfrom
what

Conversation

@spalmurray
Copy link
Copy Markdown
Collaborator

@spalmurray spalmurray commented May 12, 2025

Fix CI. Apparently there are dependencies in codecov-cli/requirements.txt that are not installed by pip install -e codecov-cli and are required for CI to run properly. This makes some sense, like dev-dependencies in js. I removed the explicit install of requirements.txt as I thought it was redundant with pip install -e codecov-cli.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 12, 2025

❌ 10 Tests Failed:

Tests completed Failed Passed Skipped
7010 10 7000 60
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.

@spalmurray spalmurray changed the title What is happening fix: CI being broken May 14, 2025
@spalmurray spalmurray merged commit d059d7f into main May 14, 2025
27 checks passed
@spalmurray spalmurray deleted the what branch May 14, 2025 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants