R1j1t / contextualSpellCheck

✔️Contextual word checker for better suggestions
MIT License
409 stars 66 forks source link

Missing requirements #18

Closed AlvaroCavalcante closed 4 years ago

AlvaroCavalcante commented 4 years ago
Lucs1590 commented 4 years ago

@R1j1t , you can improve your code if you run flake 8 .. I ran in my machine and I found pending modifications.

To more information, access: https://flake8.pycqa.org/en/latest/

R1j1t commented 4 years ago

Apart from the requirements.txt everything else looks good. I will merge it once you make the changes

R1j1t commented 4 years ago

@Lucs1590 it is a good suggestion. I thought about adding both flake8 and black when I started the project. While I was developing the features I found correcting flake8 warnings was slowing the coding process, so I stuck only with black. I understand it is a good practice and might add it as in when the project matures