Skip to content

feat: move codecov-cli to uv to simplify dependency management and build APIs#28

Merged
spalmurray merged 27 commits into
mainfrom
spalmurray/buildies
May 20, 2025
Merged

feat: move codecov-cli to uv to simplify dependency management and build APIs#28
spalmurray merged 27 commits into
mainfrom
spalmurray/buildies

Conversation

@spalmurray
Copy link
Copy Markdown
Collaborator

@spalmurray spalmurray commented May 20, 2025

I did try out uv workspaces to manage both with uv commands from the project root, but I found it a bit blah without totally changing the project structure. We can revisit that later if we want. For now the output of this is moving codecov-cli to uv for dependency management and build.

README overhaul to come later with documentation of the commands here. Just trying to get our builds published first.

Next PR will update the build and publish workflows, resolving the issue we ran into on thursday.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2025

❌ 10 Tests Failed:

Tests completed Failed Passed Skipped
6870 10 6860 10
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 force-pushed the spalmurray/buildies branch from b64aa9a to a07625d Compare May 20, 2025 16:59
@spalmurray spalmurray changed the title wip: build feat: move codecov-cli to uv to simplify dependency management and build apis May 20, 2025
@spalmurray spalmurray changed the title feat: move codecov-cli to uv to simplify dependency management and build apis feat: move codecov-cli to uv to simplify dependency management and build APIs May 20, 2025
@spalmurray spalmurray requested review from joseph-sentry and thomasrockhu-codecov and removed request for joseph-sentry May 20, 2025 18:00
Copy link
Copy Markdown
Collaborator

@thomasrockhu-codecov thomasrockhu-codecov left a comment

Choose a reason for hiding this comment

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

can you add pip install uv or equivalent to the ./install.sh script?

@spalmurray spalmurray merged commit c35d932 into main May 20, 2025
27 of 28 checks passed
@spalmurray spalmurray deleted the spalmurray/buildies branch May 20, 2025 22:07
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.

2 participants