feat: make callback return value optional in forEachBlock function - #1081

This pull request introduces an enhancement to the forEachBlock function by making the callback return value optional. Previously, the forEachBlock function expected the callback to return a value that is explicitly checked against false to determine whether to break out of the loop. With this update, the callback return value becomes optional, and the loop will continue to the next block if the callback returns undefined or true.

Changes:

1. Modified the forEachBlock function logic:

   if (callback(block) === false) {
     return false;
   }

   has been altered to handle optional return values more gracefully.

2. Updated Usage:

   • If the callback returns false, the function will still exit early.
   • If the callback returns undefined or true, the function will continue iterating through the blocks.

Reasons for Change:

• Flexibility: This change provides more flexibility in using the forEachBlock function by not forcing callbacks to always return a value.
• Backward Compatibility: Existing code that relies on the callback returning false to break the loop will continue to function as expected. Hence, this is a non-breaking change.
• Cleaner Code: Allows for cleaner callback implementations where returning a value is unnecessary.

Example Usage:

Before:

forEachBlock(block, (block) => {
  // some logic
  return true; // or return false explicitly
});

After:

forEachBlock(block, (block) => {
  // some logic
  // no need to return true explicitly
});