- 
 
- 
  Notifications
 You must be signed in to change notification settings 
- Fork 1.6k
 Replacing TS any by unknown, as using any is highly NOT recommended
 #3661
 
 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
 
 Replacing TS any by unknown, as using any is highly NOT recommended
 
 #3661
 Conversation
@raclim, you might want to check with your dev team if they agree with my proposed changes. I personally NEVER use any, unless under some very exceptional circumstances.
Thanks for the suggestion!
Will tag @clairep94 and @khanniie to chime in on this, since they have been leading the TypeScript migration work!
if typechecks pass idm merging it in! defer to @clairep94 on if she had a reason for using one over the other, i recall we may have had a conversation on a PR about using any but i forget if it was about this particular case and were we ended up on it
I'm going to merge this in since it makes sense to me, but please feel free to follow up if any thoughts come up!
Replacing TS
anybyunknown, as usinganyis highly NOT recommendedChanges:
I have verified that this pull request:
npm run lint)npm run test)developbranch.