IndieCoderMM / IndieCoderMM/todo-app

Code Quality Check

Open
#5 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
1
Forks
0
PR merge metrics
No merged PRs in 30d

Description

# Required changes for better code quality
- It will be better to declare all variables at the top to avoid unwanted re-declarations. ([index.js#L72-73](https://github.com/IndieCoderMM/todo-app/blob/master/src/index.js#L72-L73))
- All modules should be in the `modules/` folder to provide a well-organized project.
- Functions should do only one thing. Therefore, these conditions should be checked before calling the functions. ([task-manager.js#L10](https://github.com/IndieCoderMM/todo-app/blob/master/src/task-manager.js#L10), [task-manager.js#L18](https://github.com/IndieCoderMM/todo-app/blob/master/src/task-manager.js#L18), [update-status.js#L2](https://github.com/IndieCoderMM/todo-app/blob/master/src/update-status.js#L2))
- Code will be shorter and cleaner if `forEach` loops are used instead of regular `for` loops. ([task-manager.js#L11](https://github.com/IndieCoderMM/todo-app/blob/master/src/task-manager.js#L11))

Contributor guide

No contributing guide indexed for this repository

Research direction

Review src/index.js, src/task-manager.js, and src/update-status.js, beginning with the linked lines and the current module layout. Apply the requested variable, module-organization, function-responsibility, and loop-style changes across the named files. Done means the listed code-quality concerns are addressed without changing the task app's behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
web-dev
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.