fix: remove unnecessary @ts-ignore for lucide-react import#478
fix: remove unnecessary @ts-ignore for lucide-react import#478narutamaaurum wants to merge 1 commit into
Conversation
- lucide-react includes its own TypeScript definitions - The @ts-ignore comment was unnecessary and could be replaced with @ts-expect-error per lint rule - Since there's no actual TS error, simply remove the comment
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
👋 Thanks for opening a PR, @narutamaaurum!Your PR has entered the 🚦 PR Review Pipeline.
What happens next
A pipeline status comment will appear below and update automatically as your PR progresses. While you wait
This comment is posted only once. |
Summary
@ts-ignorecomment forlucide-reactimport inCodeTimeTravel.tsxlucide-reactincludes its own TypeScript definitions, so no type suppression is neededChanges
components/CodeTimeTravel.tsx: Removed@ts-ignorecomment (addresses lint warning and issue Maintenance: Upgrade@ts-ignoreto@ts-expect-errorin Playground Actions #295)Test Plan
Closes #295