Skip to content

chore: Remove publish: false in release flow#16

Merged
spalmurray merged 2 commits into
mainfrom
spalmurray/disable-dryrun
May 15, 2025
Merged

chore: Remove publish: false in release flow#16
spalmurray merged 2 commits into
mainfrom
spalmurray/disable-dryrun

Conversation

@spalmurray
Copy link
Copy Markdown
Collaborator

@spalmurray spalmurray commented May 12, 2025

Enabled the release step for our release workflows. Also moves over the last workflow (broadcast_api_changes.yml) and rolls release back to 10.4.0 so we can do the release flow again for real.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 12, 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 changed the title Remove publish: false for testing workflows, also rollback 10.4.1 so we can go again chore: Remove publish: false in release flow May 12, 2025
@spalmurray spalmurray force-pushed the spalmurray/disable-dryrun branch from 485101d to f5bf509 Compare May 12, 2025 15:05
@spalmurray spalmurray force-pushed the spalmurray/disable-dryrun branch from f5bf509 to f57e851 Compare May 15, 2025 12:47
@spalmurray spalmurray merged commit 09fa8e1 into main May 15, 2025
27 checks passed
@spalmurray spalmurray deleted the spalmurray/disable-dryrun branch May 15, 2025 13:10
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