Skip to content

Conversation

@sumaiazaman
Copy link
Contributor

@sumaiazaman sumaiazaman commented Nov 26, 2025

Currently, there are places where we check count($this->processes) === 0 directly. However, the class already has a dedicated method isEmpty() that performs the same check:

public function isEmpty()
{
    return count($this->processes) === 0;
}

This PR updates the code to use isEmpty() instead of direct count() checks. This improves readability and ensures consistency, as the isEmpty() method is already used in multiple places.

@sumaiazaman sumaiazaman changed the title [12.x] Use isEmpty() method instead of redundant count() checks in FakeProcessSequence [13.x] Use isEmpty() method instead of redundant count() checks in FakeProcessSequence Nov 26, 2025
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