New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Refactored code to fix nesting in src/user/delete.js #80

Open

Mohtelsayed wants to merge 1 commit into CMU-17313Q:f24 from Mohtelsayed:fix-nesting-src/user/delete.js

Mohtelsayed commented Sep 4, 2024

This pull request resolves #76 as it will fix 5-level function declaration nesting in function deleteUserFromFollowers in src/user/delete.js.

Changes were tested using the npm run lint and npm run test commands.


          Fix 5-level function declaration nesting in function deleteUserFromFo…

2add319

…llowers in src/user/delete.js

sonarcloud bot commented Sep 4, 2024

Quality Gate passed

Issues
0 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

coveralls commented Sep 4, 2024

Pull Request Test Coverage Report for Build 10695942460

Details

2 of 6 (33.33%) changed or added relevant lines in 1 file are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.01%) to 82.683%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/user/delete.js	2	6	33.33%

Files with Coverage Reduction	New Missed Lines	%
src/user/delete.js	1	83.19%
src/meta/errors.js	1	83.72%

Totals
Change from base Build 10550029021:	0.01%
Covered Lines:	22328
Relevant Lines:	25585

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet