Hey guys, I just pushed a new commit on the dev
branch. Can you have a look and tell me what you think?
Sure Paweł, let me pull the latest updates.
I'm on it too 👊
I see you've refactored the calculateStatistics
function. The code is much cleaner now.
Yes, I thought it was getting a bit cluttered.
The commit message is descriptive, too. Good job on mentioning the issue number it fixes.
I noticed you added some new dependencies in the package.json
. Did you also update the README
with the setup instructions?
Oops, I forgot. I'll add that right away.
I see a couple of edge cases we might not be handling in the calculateStatistic
function. Should I open an issue for that?
Yes, Bob. Please do. We should not forget to handle those.
Alright, once the README
is updated, I'll merge this commit into the main branch. Nice work, Paweł.
Thanks, @everyone! 🙌
typing