Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix multioutput requiring same number of no_update. #2987

Merged
merged 4 commits into from
Sep 6, 2024
Merged

Fix multioutput requiring same number of no_update. #2987

merged 4 commits into from
Sep 6, 2024

Conversation

T4rk1n
Copy link
Contributor

@T4rk1n T4rk1n commented Sep 5, 2024

Fixes #2986

ndrezn
ndrezn approved these changes Sep 5, 2024
View reviewed changes
Copy link
Member

@ndrezn ndrezn left a comment

Choose a reason for hiding this comment

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

Nice test 😉

@gvwilson gvwilson added sev-2 serious problem P1 needed for current cycle fix fixes something broken labels Sep 5, 2024
@T4rk1n T4rk1n merged commit 354d1a0 into dev Sep 6, 2024
3 checks passed
@T4rk1n T4rk1n deleted the fix/num-no-output-2986 branch September 6, 2024 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gvwilson gvwilson gvwilson approved these changes

@ndrezn ndrezn ndrezn approved these changes

@alexcjohnson alexcjohnson Awaiting requested review from alexcjohnson alexcjohnson is a code owner

@emilykl emilykl Awaiting requested review from emilykl emilykl is a code owner

Assignees

@T4rk1n T4rk1n

Labels
fix fixes something broken P1 needed for current cycle sev-2 serious problem
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[REGRESSION] Number of no_output must match number of outputs in dash==2.18.0
3 participants
@T4rk1n @gvwilson @ndrezn