-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
Hello @inspecta, amazing work with this project 👏 👍
I have noticed the following JavaScript good practices;
- No script tags
- Type checks are okay
- Node modules are in the .gitignore file
- No console.logs, no alerts, no document.write or eval()
-No commented out code in the repo
A few bad practices noticed include;
- DRY. Some pieces of code were repeated
https://github.com/inspecta/todo-list-webpack/blob/main/src/modules/addTask.js#L14-L31
Metadata
Metadata
Assignees
Labels
No labels