Skip to content

Use class instead of static variables for the speed measurement - #168

Merged
danielbachhuber merged 1 commit into
wp-cli:masterfrom
pfefferle:fix/144
Nov 16, 2023
Merged

Use class instead of static variables for the speed measurement #168
danielbachhuber merged 1 commit into
wp-cli:masterfrom
pfefferle:fix/144

Conversation

@pfefferle

Copy link
Copy Markdown
Contributor

this change should fix #144

@pfefferle
pfefferle requested a review from a team as a code owner November 16, 2023 10:31

@danielbachhuber danielbachhuber left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The fix works great, thanks!

Before

$ wp --require=test.php test multiple_progress_bars
1st pass  100% [=========================================================================================================================] 0:10 / 0:10
2nd pass  100% [=========================================================================================================================] 0:10 / 0:21
3rd pass  100% [=========================================================================================================================] 0:10 / 0:32

After

$ wp --require=test.php test multiple_progress_bars
1st pass  100% [=========================================================================================================================] 0:10 / 0:10
2nd pass  100% [=========================================================================================================================] 0:10 / 0:10
3rd pass  100% [=========================================================================================================================] 0:10 / 0:10
@danielbachhuber danielbachhuber added this to the 0.11.22 milestone Nov 16, 2023
@danielbachhuber
danielbachhuber merged commit 8ffd0cf into wp-cli:master Nov 16, 2023
@pfefferle
pfefferle deleted the fix/144 branch November 16, 2023 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

2 participants