search

appsmithorg / appsmith

Platform to build admin panels, internal tools, and dashboards. Integrates with 25+ databases and any API.
https://www.appsmith.com
Apache License 2.0
33.79k stars 3.63k forks source link

(JS-0323) Detected usage of the `any` type #7938

Open rishabhrathod01 opened 2 years ago

rishabhrathod01 commented 2 years ago

Description

The any type can sometimes leak into your codebase. TypeScript compiler skips the type checking of the any typed variables, so it creates a potential safety hole, and source of bugs in your codebase. We recommend using unknown or never type variable.

Occurrences

There are 1648 occurrences of this issue in the repository.

See all occurrences on DeepSource → deepsource.io/gh/appsmithorg/appsmith/issue/JS-0323/occurrences/

evergreenx commented 2 years ago

can i jump on this issue

sbalaji1192 commented 2 years ago

Hey @evergreenx ! I've assigned this issue to you, please make sure to read our Contribution Guidelines before raising a PR!

sbalaji1192 commented 2 years ago

@evergreenx After having a second look at the occurrences, I feel, we should break the issue into smaller manageable parts. Hence reassigning this. I would request you to please look at some of our other Good First Issues

bharat-patodi commented 2 years ago

@Rishabh-Rathod Can you please assign this to me? I could split it into smaller chunks like how you mentioned over in this comment?

rishabhrathod01 commented 2 years ago

Sure @bharat-patodi, that would work here also. 20-30 Occurrences should be more than enough for a single PR, less will also do.