Skip to content

Conversation

@thomasrockhu-codecov
Copy link
Contributor

No description provided.

@sentry
Copy link

sentry bot commented Sep 19, 2025

❌ 3 Tests Failed:

Tests completed Failed Passed Skipped
9 3 6 0
View the top 3 failed test(s) by shortest run time
app/test_calculator.py::test_fail_1
Stack Traces | 0.001s run time
def test_fail_1():
>       assert True is False
E       assert True is False

app/test_calculator.py:11: AssertionError
app/test_calculator.py::test_fail_2
Stack Traces | 0.001s run time
def test_fail_2():
>       assert True is False
E       assert True is False

app/test_calculator.py:14: AssertionError
app/test_calculator.py::test_fail_3
Stack Traces | 0.001s run time
def test_fail_3():
>       assert True is False
E       assert True is False

app/test_calculator.py:17: 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.

Comment on lines 21 to 24
run: |
pip install sentry-prevent-cli
sentry-prevent-cli -v upload --report-type test-results
uses: getsentry/prevent-action@v0
env:

Choose a reason for hiding this comment

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

Potential bug: The CI workflow step incorrectly mixes run: and uses:, which is invalid syntax and will cause the workflow to fail.
  • Description: The CI workflow file .github/workflows/ci.yml contains a step that attempts to use both run: | and uses: getsentry/prevent-action@v0 simultaneously. According to GitHub Actions syntax, a single step cannot contain both run and uses keywords. This syntax error will prevent the workflow from being parsed correctly, causing the entire CI pipeline to fail on every run. The error was likely introduced during a migration from a command-line tool to a dedicated GitHub Action.

  • Suggested fix: Remove the run: | line from the "Upload test results to Codecov" step. The uses: getsentry/prevent-action@v0 key should be at the same indentation level as the name and if keys for the step to be syntactically correct.
    severity: 0.9, confidence: 1.0

Did we get this right? 👍 / 👎 to inform future reviews.

@thomasrockhu-codecov thomasrockhu-codecov merged commit 014b906 into main Sep 19, 2025
3 of 5 checks passed
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