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

Reduce code #3125

Open
iower opened this issue Aug 3, 2020 · 0 comments
Open

Reduce code #3125

iower opened this issue Aug 3, 2020 · 0 comments
Labels
code health draft Most likely, there will be changes in the issue. Not urgent or necessary for now. It needs planning

Comments

@iower
Copy link
Collaborator

iower commented Aug 3, 2020

Issue

  • There is too much code in the project
  • More code, more bugs
  • Difficult onboarding

Check

Excluding libraries, git, build, and resources (such as images, fonts...)

find \( -name '*.js' -or -name '*.jsx' -or -name '*.json' -or -name '*.yml' -or -name '*.css' -or -name '*.scss' -or -name '*.md' -or -name '.gitignore' \) -not -path './node_modules/*' -and -not -path './.git/*' -and -not -path './build-*/*' -exec ls -l {} \; | awk '{ Total += $5} END { print Total }'-n

11690038 or 11Mb

Let the new developer read the code at 10b/sec,

=> 1Msec => 280h => 35 working days to read (only reading, not understanding, without libs and domain study)

Task

  • Carefully remove all obsolete code
  • Simplify code
  • Goal: 1Mb

Task parts

Done

In the process

Planned

  • Push: auto-measure & notify
  • ???
@iower iower added draft Most likely, there will be changes in the issue. Not urgent or necessary for now. It needs planning code health labels Aug 3, 2020
@Disti4ct Disti4ct mentioned this issue May 22, 2021
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code health draft Most likely, there will be changes in the issue. Not urgent or necessary for now. It needs planning
Projects
None yet
Development

No branches or pull requests

1 participant