Page MenuHomePhabricator

Remove the unused code
Open, Needs TriagePublic

Description

My console after starting the server gives the following result. Remove the unwanted code, test the tool and create a patch :-)

./src/components/home.js
  Line 1:28:    'useContext' is defined but never used                 no-unused-vars
  Line 151:33:  'rest' is assigned a value but never used              no-unused-vars
  Line 919:13:  'uploadedFile' is assigned a value but never used      no-unused-vars
  Line 920:11:  'uploadProperties' is assigned a value but never used  no-unused-vars

./src/components/Header.jsx
  Line 3:10:  'Layout' is defined but never used    no-unused-vars
  Line 3:26:  'Menu' is defined but never used      no-unused-vars
  Line 3:44:  'Dropdown' is defined but never used  no-unused-vars

./src/App.js
  Line 3:32:  'Router' is defined but never used  no-unused-vars

Event Timeline

Change 668254 had a related patch set uploaded (by Sandyabhi; owner: Sandyabhi):
[labs/tools/VideoCutTool@master] Removed the unused code

https://gerrit.wikimedia.org/r/668254

Change 668254 merged by jenkins-bot:
[labs/tools/VideoCutTool@master] Removed the unused code

https://gerrit.wikimedia.org/r/668254

Hey ! I would like to solve this issue please assign it to me and yes please tell me from where to get the code base to get started as early as possible and remove all the unused codes😄

Gopavasanth added a subscriber: Sandyabhi.

@Gaurav24072002 Thanks for your interest! I think @Sandyabhi made most of the work on this task! If you still find any unused code please feel free to make a patch!

Check: https://phabricator.wikimedia.org/project/profile/4054/ for the code repositories :)

Change 673501 had a related patch set uploaded (by Sandyabhi; owner: Sandyabhi):
[labs/tools/VideoCutTool@master] Remove unused code from home.js

https://gerrit.wikimedia.org/r/673501

Change 673501 abandoned by Sandyabhi:
[labs/tools/VideoCutTool@master] Remove unused code from home.js

Reason:

https://gerrit.wikimedia.org/r/673501

Hey , @Gopavasanth I have found few unused function parameters in home.js. Should I create a patch ?