Skip to content

Make which(on_error change backward compatible with log_error=False#3664

Merged
smoors merged 1 commit intoeasybuilders:developfrom
Flamefire:enhance_which
Apr 28, 2021
Merged

Make which(on_error change backward compatible with log_error=False#3664
smoors merged 1 commit intoeasybuilders:developfrom
Flamefire:enhance_which

Conversation

@Flamefire
Copy link
Copy Markdown
Contributor

Fix for #3661 which led to log_error=False getting ignored

Uses None now to detect if the new param was set.

@smoors

Copy link
Copy Markdown
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

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

lgtm

@smoors smoors added the bug fix label Apr 28, 2021
@smoors smoors added this to the next release (4.3.5?) milestone Apr 28, 2021
@smoors
Copy link
Copy Markdown
Contributor

smoors commented Apr 28, 2021

Going in, thanks @Flamefire!

@smoors smoors merged commit 4c41040 into easybuilders:develop Apr 28, 2021
@Flamefire Flamefire deleted the enhance_which branch April 28, 2021 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants