You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This codebase has changed a lot since the first commit. @sam33rdhakal and myself will undertake a basic audit, looking for the following things:
Code we don't understand
Duplicated code
Unused code
Over-complicated code
We'll create PRs for these as we see them, and link them back to this issue. Note that we are not looking to make any significant changes to business logic; if they do get proposed, such changes will be highlighted.
You can follow along with our progress in the branch code_audit; I'm adding comments and questions in the branch pc-identify-codebase-problems-2018.
The text was updated successfully, but these errors were encountered:
This codebase has changed a lot since the first commit. @sam33rdhakal and myself will undertake a basic audit, looking for the following things:
We'll create PRs for these as we see them, and link them back to this issue. Note that we are not looking to make any significant changes to business logic; if they do get proposed, such changes will be highlighted.
You can follow along with our progress in the branch
code_audit
; I'm adding comments and questions in the branchpc-identify-codebase-problems-2018
.The text was updated successfully, but these errors were encountered: