javascript-algorithms icon indicating copy to clipboard operation
javascript-algorithms copied to clipboard

Changed the BinarySearch component from a function to an arrow function.

Open Ben-lad opened this issue 2 years ago • 4 comments

Changed the BinarySearch component from a function to an arrow function.

The original component was defined using the function keyword, but I updated it to use an arrow function instead. This change provides a more concise syntax and also avoids any issues with the binding of the "this" keyword.

I have tested the updated component and verified that it works as expected.

Thanks for considering this change!

Ben-lad avatar Feb 20 '23 00:02 Ben-lad

It's a top-level function. There's no issue with "this", nor is it more concise. Also it's not a component.

Consider common sense before spamming multiple repositories with the same artificial-not-so-intelligent type of changes.

lazarljubenovic avatar Feb 20 '23 05:02 lazarljubenovic

It's a top-level function. There's no issue with "this", nor is it more concise. Also it's not a component.

Consider common sense before spamming multiple repositories with the same artificial-not-so-intelligent type of changes.

don't that hard on the guy, he might still be learning and just wants to contribute.

iamleomoko avatar Mar 15 '23 16:03 iamleomoko

I looked at his profile; he had the same comments in other repos. It's a bot.

lazarljubenovic avatar Mar 15 '23 16:03 lazarljubenovic

@lazarljubenovic I'm really sorry for that, if that make you angry. I'm not a bot just still learning as @iamleomoko said, and am looking for something small to change in the code. I'll delete this now So am really sorry once again.

@iamleomoko Thank you for saying that.

Ben-lad avatar Mar 16 '23 11:03 Ben-lad