Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add check for any in BE #133

Closed
littlewhywhat opened this issue Mar 20, 2020 · 2 comments · Fixed by #144
Closed

Add check for any in BE #133

littlewhywhat opened this issue Mar 20, 2020 · 2 comments · Fixed by #144

Comments

@littlewhywhat
Copy link
Contributor

Right now there is no error if we give 'any' as a parameter to the type-safe function.

One of possible options:
https://stackoverflow.com/a/32228674

@littlewhywhat
Copy link
Contributor Author

littlewhywhat commented Mar 31, 2020

noImplicitAny doesn't feel it - you still can do something like this:
`
const foo = (a: string) { console.log(a) }

let a: any

foo(a)
`
and have no errors

littlewhywhat added a commit that referenced this issue Mar 31, 2020
littlewhywhat added a commit that referenced this issue Mar 31, 2020
@littlewhywhat
Copy link
Contributor Author

littlewhywhat commented Mar 31, 2020

and the issue originated from here #113 (comment) where in express.Request - body is mentioned as any... so this case will have to be caught in review I guess

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant