Skip to content

Chore/remove unused variables forgemode#1387

Open
Abhishek2005-ard wants to merge 19 commits into
aryandas2911:mainfrom
Abhishek2005-ard:chore/remove-unused-variables-forgemode
Open

Chore/remove unused variables forgemode#1387
Abhishek2005-ard wants to merge 19 commits into
aryandas2911:mainfrom
Abhishek2005-ard:chore/remove-unused-variables-forgemode

Conversation

@Abhishek2005-ard
Copy link
Copy Markdown
Contributor

Description

This PR addresses ESLint warnings (no-unused-vars) in ForgeMode.jsx by cleaning up code that was declared but never utilized. Removing these unused references helps keep the codebase tidy and slightly reduces the memory footprint of the component.

Changes Made

  • Removed routinesList state: The component fetched routines and stored them in routinesList, but this state was never accessed for rendering or logic. The data is now accessed locally where needed without saving it to an unused React state.
  • Removed getBorderGlowClass function: This helper function was defined to generate dynamic CSS classes, but it was never invoked within the JSX return block.

Testing

  • Verified that npm run lint now passes without any unused variable warnings.
  • Confirmed that Forge Mode functionality remains unaffected by the removal of these variables.

close #1384

@Abhishek2005-ard
Copy link
Copy Markdown
Contributor Author

@aryandas2911
Pls review this pr

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

Successfully merging this pull request may close these issues.

Clean up unused variables and functions in ForgeMode

1 participant