Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clear out TODOs #321

Open
2 tasks
Tracked by #337
ceyhunsen opened this issue Sep 25, 2024 · 0 comments
Open
2 tasks
Tracked by #337

Clear out TODOs #321

ceyhunsen opened this issue Sep 25, 2024 · 0 comments
Assignees

Comments

@ceyhunsen
Copy link
Member

Issue Description

There are TODO statements everywhere. They should be removed one by one. Feel free to open an issue if any TODO needs more detailing.

To check TODO statements, either scripts/check_for_todos.sh or editor integrated search tools can be used.

Also, there is a new Github job that checks for TODO keyword. It's logs can be checked for that. Currently it's not enforced for PR's to pass but in the future it will.

Tasks

Parts of the task:

  • Clear out TODO statements
  • Enable and enforce TODO check script in Github workflow

Deadline of the Issue

Before mainnet launch

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

No branches or pull requests

3 participants