Refactoring code in src/notifications.js

CMU-17313Q / NodeBB

GNU General Public License v3.0

0 stars 50 forks source link

Refactoring code in src/notifications.js #97

Open edemenkov opened 3 months ago

edemenkov commented 3 months ago

File: src/notifications.js

Issue: Refactor this function to reduce its Cognitive Complexity from 20 to the 15 allowed.

Explanation: The function pushToUids in src/notifications.js has a Cognitive Complexity of 20 due to too many nested asynchronous calls and conditional checks, so it needs to be refactored to reduce these and bring the complexity down to the allowed limit of 15.

edemenkov commented 3 months ago

I would like to work on this please!

github-actions[bot] commented 3 months ago

Great! I assigned you (@edemenkov) to the issue. Have fun working on it!