search

CMU-17313Q / NodeBB

GNU General Public License v3.0
0 stars 50 forks source link

Changed-cognitive-complexity #95

Open Aya-Abdel-Hamid opened 2 months ago

Aya-Abdel-Hamid commented 2 months ago

This update resolves several linting errors and import path issues in the groups.js file. The self-import of the groups module was removed to fix the "module imports itself" error. Additionally, the unresolved path issues for the user and logGroupEvent imports were addressed by correcting the relative paths and ensuring proper module resolution. The logGroupEvent function, previously misidentified as being imported from a non-existent file, is now correctly imported from the ../utils module where it is defined as a shared utility function. Though I fixed the above mentioned errors, the code still fails some lint tests on github.

sonarcloud[bot] commented 2 months ago

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud