-
-
Notifications
You must be signed in to change notification settings - Fork 98
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
Update/Remove Todos in the 'Codebase Best Practices' page #393
Comments
in first image, it looks like we are already implementing the practice mentioned. |
those would have been invisible comments, they worked with the previous build, now the review of the various pages is a WIP the ideal would be to check if the comments still need to be there, or if they are actionable. If they are actionable, do the thing and remove them, if they are no longer relevant remove them. |
OK, If the comments are no longer relevant, I will simply remove them. |
@shyam-cmd I don't know if they are not relevant, you would need to check the status of the things mentioned |
Describe the issue
In the https://contribute.freecodecamp.org/codebase-best-practices/ page, there are some todos/comments that should be remove/updated
I think for the 1st image, we're doing it sort of, not sure about the second image, since we don't have a store
The text was updated successfully, but these errors were encountered: