Removed all todos.

[ranbir7]

What does this PR do?

Fixes #1484

I removed the todos

Before submitting

• Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
• [yes ] Did you read the contributor guideline, Pull Request section?
• [ yes] Did you make sure to update the docs?
• [no ] Did you write any new necessary tests?

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

[Borda]

Hi, any reason why you are removing all; there are a few understands if issues to be fixed...

[codecov]

Codecov Report

Merging #1489 (875a2b5) into master (debd53c) will increase coverage by 0%.
The diff coverage is n/a.

Additional details and impacted files

@@          Coverage Diff           @@
##           master   #1489   +/-   ##
======================================
  Coverage      90%     90%           
======================================
  Files         213     213           
  Lines       10921   10921           
======================================
+ Hits         9793    9794    +1     
+ Misses       1128    1127    -1     

[ranbir7]

The issue stated that we have to remove each todo one by one
So I did it
Please guide me though
I'm just very new to this

[Borda]

The issue stated that we have to remove each todo one by one So I did it Please guide me though I'm just very new to this

sure, so pls remove just one line with todo and run pre-commit --all-files and fix all raised issue :)