Skip to content
This repository has been archived by the owner on Mar 7, 2024. It is now read-only.

[qa] Commented out sections of Code #109

Open
pandafy opened this issue Sep 30, 2020 · 7 comments
Open

[qa] Commented out sections of Code #109

pandafy opened this issue Sep 30, 2020 · 7 comments
Assignees

Comments

@pandafy
Copy link
Contributor

pandafy commented Sep 30, 2020

I have seen commented out sections of code in multiple files. I don't understand their purpose as they are just code and does not conveys any useful information. In my opinion, it will be better to remove them to improve code quality.

@pandafy pandafy changed the title Commented out sections of Code [qa] Commented out sections of Code Sep 30, 2020
@prichoms
Copy link

prichoms commented Oct 1, 2020

can i work upon it?

@SumaiyaSafdar
Copy link

hey @Abhishek-kumar09 can I work on this??

@Abhishek-kumar09
Copy link
Contributor

@SumaiyaSafdar yes you can start working on it.

@SumaiyaSafdar
Copy link

hey @Abhishek-kumar09 can you please help me the basics?
how to start? what is my work

@Abhishek-kumar09
Copy link
Contributor

@SumaiyaSafdar You can go through different files inside the src/ directory and look for the comments that are not adding any value to the code, like only the commented out codes.

See comments are meant to add some description to the code written, but many a times, we left the comments that are not useful for the future, like (commenting code that is not working just to start the server, etc). You have to delete those comments and make a PR.

You can start looking at files and see where are the comments if the comments are useful leave them as it is, if you think it is not useful delete them, that's it.

@SumaiyaSafdar
Copy link

Thank you so much @Abhishek-kumar09 ,I'll get back to you asap ,it's an easy work. A very big thanks for your help.

@SumaiyaSafdar
Copy link

@Abhishek-kumar09 I've created a PR .Please look to it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants