Skip to content

chore: remove unused imports in layout.tsx and Hero.tsx (#676)#691

Open
kumar-ayan wants to merge 1 commit into
devpathindcommunity-india:masterfrom
kumar-ayan:fix/issue-676/remove-unused-imports
Open

chore: remove unused imports in layout.tsx and Hero.tsx (#676)#691
kumar-ayan wants to merge 1 commit into
devpathindcommunity-india:masterfrom
kumar-ayan:fix/issue-676/remove-unused-imports

Conversation

@kumar-ayan

Copy link
Copy Markdown
Contributor

Description

Removed unused imports that were cluttering code and could cause lint failures:

  • src/app/layout.tsx
    • removed import BackgroundMesh from '@/components/layout/BackgroundMesh'; (the only reference was a commented-out JSX line)
    • removed import PageTrackerInit from '@/components/PageTrackerInit'; (unused)
  • src/components/home/Hero.tsx
    • removed import Image from 'next/image'; (unused)
    • removed import { MagneticText } from '../ui/magnetic-text'; (unused)

These changes keep the codebase clean and align with linting best practices.

Fixes #676

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • Ran project linting to confirm no unused-import warnings/errors remain for the modified files:

    • npm run lint
  • Built the project locally to ensure no build-time errors introduced by removing the imports:

    • npm run build
  • Verified the app starts without issues in local dev mode (spot-checked the affected pages/components)

  • Local testing

  • Vercel Preview Deployment

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings or errors
  • I have checked that the "DevPath India" branding remains intact

@github-actions github-actions Bot added gssoc26 This is a official GirlScript Summer of Code label. level:beginner Beginner level issues type:bug type:refactor labels Jun 21, 2026
@kumar-ayan

Copy link
Copy Markdown
Contributor Author

The CI failure isn't from my changes—this PR only removes unused imports from 2 files (+1 -6 lines). The 306 ESLint errors are pre-existing issues in other files (ShareButton.tsx, AuthContext.tsx, useBookmarks.ts, useGamification.ts, useMaintenance.ts, github.ts, three.d.ts). Happy to help fix those in a follow-up PR if needed!

@kumar-ayan

Copy link
Copy Markdown
Contributor Author

Hey @Aditya948351 , @devpathindcommunity-india , can you check this pr out?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc26 This is a official GirlScript Summer of Code label. level:beginner Beginner level issues type:bug type:refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Unused imports in layout.tsx and Hero.tsx

1 participant