Skip to content

fix: remove unused CodeQL imports#25

Merged
sbolel merged 1 commit into
mainfrom
fix/codeql-unused-imports
May 31, 2026
Merged

fix: remove unused CodeQL imports#25
sbolel merged 1 commit into
mainfrom
fix/codeql-unused-imports

Conversation

@sbolel

@sbolel sbolel commented May 31, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove the unused MarkdownHighlighter import from src/App.tsx
  • remove the unused useAnimation import from src/components/CursorSparkles.tsx

Validation

  • npm ci --cache /private/tmp/paste-to-markdown-npm-cache
  • npm run typecheck
  • npm run build
  • npm run lint
  • rg "MarkdownHighlighter|useAnimation" src
  • rg "import { MarkdownHighlighter } from '@/components/MarkdownHighlighter'|useAnimation" src/App.tsx src/components/CursorSparkles.tsx

@sbolel sbolel requested a review from Copilot May 31, 2026 08:58
@sbolel sbolel self-assigned this May 31, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes unused imports that were flagged by CodeQL/linting, reducing dead code and keeping the import surface minimal.

Changes:

  • Removed unused useAnimation import from CursorSparkles.
  • Removed unused MarkdownHighlighter import from App.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/components/CursorSparkles.tsx Drops unused useAnimation import from framer-motion.
src/App.tsx Drops unused MarkdownHighlighter import.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sbolel sbolel merged commit 93f7680 into main May 31, 2026
6 checks passed
@sbolel sbolel deleted the fix/codeql-unused-imports branch May 31, 2026 09:07
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants