Skip to content

feat: create new upload command#77

Merged
joseph-sentry merged 5 commits intomainfrom
joseph/fix-cmds
Jul 22, 2025
Merged

feat: create new upload command#77
joseph-sentry merged 5 commits intomainfrom
joseph/fix-cmds

Conversation

@joseph-sentry
Copy link
Copy Markdown
Contributor

@joseph-sentry joseph-sentry commented Jul 16, 2025

Fixes CCMRG-1362
Fixes CCMRG-1361

we wanted to have a simplified interface so we're going to create a new "upload" command.

This command is just a copy of upload-coverage but obviously with a diffrent name and it also accepts test-results alongside test_results for the "--report-type" option

we wanted to have a simplified interface so we're going to create a new "upload"
command.

This command is just a copy of upload-coverage but obviously with a diffrent
name and it also accepts `test-results` alongside `test_results` for the
"--report-type" option
@joseph-sentry joseph-sentry requested a review from spalmurray July 16, 2025 21:13
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 16, 2025

❌ 10 Tests Failed:

Tests completed Failed Passed Skipped
6890 10 6880 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.

if report_type_str == "coverage":
return ReportType.COVERAGE
elif report_type_str == "test_results":
elif report_type_str.replace("-", "_") == "test_results":
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lol nice

@spalmurray
Copy link
Copy Markdown
Collaborator

u gotta run the make command_dump thing

@linear
Copy link
Copy Markdown

linear bot commented Jul 17, 2025

@joseph-sentry joseph-sentry merged commit 327cbef into main Jul 22, 2025
26 checks passed
@joseph-sentry joseph-sentry deleted the joseph/fix-cmds branch July 22, 2025 20:13
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