chore: replace @ts-ignore with @ts-expect-error in playground actions#397
Conversation
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? |
👋 Thanks for opening a PR, @CodesWithSubham!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. |
|
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:
WalkthroughChanged the TypeScript suppression directive in modules/playground/actions/index.ts (line 249) from ChangesTypeScript Error Suppression Best Practice
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
Replace the existing `@ts-ignore` directive in Playground Actions with `@ts-expect-error` to follow TypeScript best practices. This ensures the compiler will report an error if the ignored TypeScript issue is resolved in the future, preventing unnecessary suppression comments from remaining in the codebase. Changes: - replaced `@ts-ignore` with `@ts-expect-error` - kept existing runtime behavior unchanged - no functional logic modified Signed-off-by: Subham Duary <subhamduary11@gmail.com>
12c3827 to
800b10d
Compare
Summary
@ts-ignoredirective with@ts-expect-errorinmodules/playground/actions/index.tsType of change
Related issue
Closes #295
Validation
npm run lintnpm testnpm run buildList any additional manual verification you performed:
Screenshots or recordings
Not applicable — this change only updates a TypeScript suppression directive and does not affect the UI.
Checklist
Summary by CodeRabbit