MrSuddenJoy / 4programmers.net

Coyote
https://4programmers.net
MIT License
0 stars 1 forks source link

refactor: fix `// @ts-ignore` to `// @ts-expect-error` #38

Closed deepsource-autofix[bot] closed 3 months ago

deepsource-autofix[bot] commented 3 months ago

TypeScript allows you to suppress all errors on a line by placing a single-line comment starting with @ts-ignore immediately before the erroring line. While powerful, there is no way to know if a @ts-ignore is actually suppressing an error without manually investigating what happens when the @ts-ignore is removed. This means its easy for @ts-ignores to be forgotten about, and remain in code even after the error they were suppressing is fixed. This is dangerous, as if a new error arises on that line it'll be suppressed by the forgotten about @ts-ignore, and so be missed.

deepsource-io[bot] commented 3 months ago

Here's the code health analysis summary for commits 4288232..3cd8a64. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource PHP LogoPHP✅ SuccessView Check ↗
DeepSource JavaScript LogoJavaScript❌ Failure
❗ 29 occurences introduced
🎯 57 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.